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