Aleksey Plekhanov created IGNITE-23237:
------------------------------------------

             Summary: Calcite engine. Refactor CalciteQueryProcessorTest
                 Key: IGNITE-23237
                 URL: https://issues.apache.org/jira/browse/IGNITE-23237
             Project: Ignite
          Issue Type: Improvement
            Reporter: Aleksey Plekhanov


CalciteQueryProcessorTest contains a lot of not grouped tests. We can split 
this class into several different classes and group tests somehow (or at least 
move some tests to the new classes). 
It also contains a lot of boilerplate code, like:
{noformat}
QueryEngine engine = Commons.lookupComponent(grid(1).context(), 
QueryEngine.class);

List<FieldsQueryCursor<List<?>>> qry = engine.query(...)
{noformat}
We can inherit CalciteQueryProcessorTest (or any new test classes) from 
AbstractBasicIntegrationTest and replace query start routine to `sql(...)`.

Some methods can be removed (like testThroughput), it doesn't called by any 
test, and it assumes that querySqlFields is executed on H2 engine (it's not 
true anymore).



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to