patwari-radhika commented on code in PR #16113:
URL: https://github.com/apache/pinot/pull/16113#discussion_r2200979286


##########
pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeTableDataManager.java:
##########
@@ -958,4 +960,19 @@ private boolean isEnforceConsumptionInOrder() {
     StreamIngestionConfig streamIngestionConfig = getStreamIngestionConfig();
     return streamIngestionConfig != null && 
streamIngestionConfig.isEnforceConsumptionInOrder();
   }
+
+  /**
+   * Sets the skipOriginalFieldInUnnest field using server level config if 
table level config is false.
+   */
+  private void setSkipOriginalFieldInUnnest(TableConfig tableConfig) {

Review Comment:
   If table config cannot be modified, then InstanceDataManagerConfig will have 
be passed all the way to TransformPipeline. Other way is to set the value of 
this field in table config in HelixInstanceDataManager.java - so that table 
config is updated before creating TableDataManager



-- 
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: commits-unsubscr...@pinot.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to