ankitsultana commented on code in PR #14598:
URL: https://github.com/apache/pinot/pull/14598#discussion_r1872232274
##########
pinot-query-runtime/src/main/java/org/apache/pinot/query/runtime/timeseries/PhysicalTimeSeriesServerPlanVisitor.java:
##########
@@ -42,17 +42,14 @@
import org.apache.pinot.tsdb.spi.plan.LeafTimeSeriesPlanNode;
-public class PhysicalTimeSeriesPlanVisitor {
- public static final PhysicalTimeSeriesPlanVisitor INSTANCE = new
PhysicalTimeSeriesPlanVisitor();
-
+public class PhysicalTimeSeriesServerPlanVisitor {
private QueryExecutor _queryExecutor;
private ExecutorService _executorService;
private ServerMetrics _serverMetrics;
- private PhysicalTimeSeriesPlanVisitor() {
- }
-
- public void init(QueryExecutor queryExecutor, ExecutorService
executorService, ServerMetrics serverMetrics) {
+ // Warning: Don't use singleton access pattern, since Quickstarts run in a
single JVM and spawn multiple broker/server
Review Comment:
The issue only manifests in Quickstarts when numServers > 1. The previous
pattern of using an INSTANCE variable leads to hard to debug errors, so left a
Warning for future readers.
--
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]
