alex-plekhanov commented on a change in pull request #8855:
URL: https://github.com/apache/ignite/pull/8855#discussion_r587472941
##########
File path:
modules/calcite/src/main/java/org/apache/ignite/internal/processors/query/calcite/schema/IgniteTableImpl.java
##########
@@ -59,16 +62,21 @@
/** */
private final Statistic statistic;
+ /** */
+ private final GridKernalContext ctx;
+
/** */
private volatile GridH2Table tbl;
/** */
private final Map<String, IgniteIndex> indexes = new ConcurrentHashMap<>();
/**
- * @param desc Table descriptor.
+ * @param ctx Kernal context.
+ * @param desc Table descriptor.
*/
- public IgniteTableImpl(TableDescriptor desc) {
+ public IgniteTableImpl(GridKernalContext ctx, TableDescriptor desc) {
Review comment:
But why? IgniteTableImpl already uses KernelContext, but obtains it from
cache context (which is not always initialized). Also, there is no other way to
get GridCacheProcessor to start lazy caches.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
