sanpwc commented on code in PR #2500:
URL: https://github.com/apache/ignite-3/pull/2500#discussion_r1319617681
##########
modules/table/src/main/java/org/apache/ignite/internal/table/distributed/TableManager.java:
##########
@@ -219,17 +200,6 @@ public class TableManager extends Producer<TableEvent,
TableEventParameters> imp
private static final int TX_STATE_STORAGE_FLUSH_DELAY = 1000;
private static final IntSupplier TX_STATE_STORAGE_FLUSH_DELAY_SUPPLIER =
() -> TX_STATE_STORAGE_FLUSH_DELAY;
- /**
- * If this property is set to {@code true} then an attempt to get the
configuration property directly from Meta storage will be skipped,
- * and the local property will be returned.
- * TODO: IGNITE-16774 This property and overall approach, access
configuration directly through Meta storage,
- * TODO: will be removed after fix of the issue.
- */
- private final boolean getMetadataLocallyOnly =
IgniteSystemProperties.getBoolean("IGNITE_GET_METADATA_LOCALLY_ONLY");
Review Comment:
Before your changes within non-sql API we've checked tables explicitly
though MS, meaning that we had metadata data sync, it's no longer true, right?
--
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]
