korlov42 commented on code in PR #2770:
URL: https://github.com/apache/ignite-3/pull/2770#discussion_r1379776802
##########
modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/ExecutionContext.java:
##########
@@ -41,16 +42,14 @@
import org.apache.ignite.internal.sql.engine.exec.mapping.ColocationGroup;
import org.apache.ignite.internal.sql.engine.exec.mapping.FragmentDescription;
import org.apache.ignite.internal.sql.engine.type.IgniteTypeFactory;
-import org.apache.ignite.internal.sql.engine.util.AbstractQueryContext;
-import org.apache.ignite.internal.sql.engine.util.BaseQueryContext;
import org.apache.ignite.internal.sql.engine.util.TypeUtils;
import org.apache.ignite.network.ClusterNode;
import org.jetbrains.annotations.Nullable;
/**
* Runtime context allowing access to the tables in a database.
*/
-public class ExecutionContext<RowT> extends AbstractQueryContext implements
DataContext {
Review Comment:
> seems AbstractQueryContext no need anymore
done
> also interface Context now looks strange
not sure which one... Elaborate please
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
