adarshsanjeev commented on code in PR #18121:
URL: https://github.com/apache/druid/pull/18121#discussion_r2178067849
##########
extensions-core/multi-stage-query/src/main/java/org/apache/druid/msq/exec/ControllerContext.java:
##########
@@ -33,17 +33,23 @@
import org.apache.druid.server.DruidNode;
import java.io.File;
+import java.util.Map;
/**
* Context used by multi-stage query controllers. Useful because it allows
test fixtures to provide their own
* implementations.
*/
public interface ControllerContext
{
+ /**
+ * Query ID for this context.
+ */
+ String queryId();
Review Comment:
The context being aware of the queryId does make sense. Now that we have
this, we could get the queryId from the context instead of having it in the
controller. Would that make it neater?
--
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]
