raghavyadav01 commented on code in PR #14093:
URL: https://github.com/apache/pinot/pull/14093#discussion_r1777891167
##########
pinot-spi/src/main/java/org/apache/pinot/spi/stream/StreamConfig.java:
##########
@@ -79,10 +78,11 @@ public class StreamConfig {
// Allow overriding it to use different offset criteria
private OffsetCriteria _offsetCriteria;
- // Indicates if the segment should be uploaded to the deep store's file
system or to the controller during the
- // segment commit protocol. By default, segment is uploaded to the
controller during commit.
- // If this flag is set to true, the segment is uploaded to deep store.
- private final boolean _serverUploadToDeepStore;
+ // Indicate StreamConfig flag for table if segment should be uploaded to the
deep store's file system or to the
+ // controller during the segment commit protocol. if config is not present
in Table StreamConfig
+ // _serverUploadToDeepStore is null and method isServerUploadToDeepStore()
overrides the default value with Server
+ // level config
+ private final String _serverUploadToDeepStore;
Review Comment:
Fixed it.
##########
pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/SegmentCommitterFactory.java:
##########
@@ -54,7 +57,13 @@ public SegmentCommitterFactory(Logger segmentLogger,
ServerSegmentCompletionProt
public SegmentCommitter
createSegmentCommitter(SegmentCompletionProtocol.Request.Params params,
String controllerVipUrl)
throws URISyntaxException {
- boolean uploadToFs = _streamConfig.isServerUploadToDeepStore();
+ InstanceDataManagerConfig instanceDataManagerConfig =
_indexLoadingConfig.getInstanceDataManagerConfig();
+ PinotConfiguration config = instanceDataManagerConfig.getConfig();
Review Comment:
Fixed.
--
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
