gortiz commented on issue #12355: URL: https://github.com/apache/pinot/issues/12355#issuecomment-1959178580
I really like this feature. What I'm not sure about is the syntax. Not because I'm against URL params. But I would like to prioritize consistency here. It doesn't seem we have other query options where the value is a `Map<String, Collection<T>>`, so adding this feature as it is would mean that future options should use this syntax. -- 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
