adarshsanjeev commented on code in PR #16572:
URL: https://github.com/apache/druid/pull/16572#discussion_r1643685742
##########
extensions-core/multi-stage-query/src/main/java/org/apache/druid/msq/querykit/results/ExportResultsFrameProcessorFactory.java:
##########
@@ -63,19 +64,22 @@ public class ExportResultsFrameProcessorFactory implements
FrameProcessorFactory
private final ExportStorageProvider exportStorageProvider;
private final ResultFormat exportFormat;
private final ColumnMappings columnMappings;
+ private final ResultsContext resultsContext;
@JsonCreator
public ExportResultsFrameProcessorFactory(
@JsonProperty("queryId") String queryId,
@JsonProperty("exportStorageProvider") ExportStorageProvider
exportStorageProvider,
@JsonProperty("exportFormat") ResultFormat exportFormat,
- @JsonProperty("columnMappings") @Nullable ColumnMappings columnMappings
+ @JsonProperty("columnMappings") @Nullable ColumnMappings columnMappings,
+ @JsonProperty("resultsContext") @Nullable ResultsContext resultsContext
Review Comment:
The field is nullable, so it should not cause any issues with backward
compatibility, 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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
