xiangfu0 commented on code in PR #16746:
URL: https://github.com/apache/pinot/pull/16746#discussion_r2324066978
##########
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotQueryResource.java:
##########
@@ -160,54 +160,67 @@ public StreamingOutput
handleTimeSeriesQueryRange(@QueryParam("language") String
@POST
@Path("validateMultiStageQuery")
- public MultiStageQueryValidationResponse
validateMultiStageQuery(MultiStageQueryValidationRequest request,
+ public List<MultiStageQueryValidationResponse>
validateMultiStageQuery(MultiStageQueryValidationRequest request,
Review Comment:
Another way is to add a new field `sqls` under
`MultiStageQueryValidationRequest` then you can set `sqls = List.of(sql)` if
called the old API.
All the processing logic could still assume this is an array of sqls
--
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]
