jackjlli commented on issue #12745:
URL: https://github.com/apache/pinot/issues/12745#issuecomment-2043525012
PR merged. We can close this issue now. Thanks everyone for working on this!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on t
jackjlli closed issue #12745: NPE when broker header is null
URL: https://github.com/apache/pinot/issues/12745
--
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
Jackie-Jiang commented on issue #12745:
URL: https://github.com/apache/pinot/issues/12745#issuecomment-2038485999
I saw we are passing `null` as `HttpHeaders` from the test. In reality,
could it ever be `null` when it is passed from the rest API stub?
--
This is an automated message from
shounakmk219 commented on issue #12745:
URL: https://github.com/apache/pinot/issues/12745#issuecomment-2029400572
@jackjlli regarding how the database name is being passed, let's use the
discussion thread
[here](https://docs.google.com/document/d/1l1dco4lRhB28y-yKMVKlOMZ0sdFKObjJfhAjG6cmktU
shounakmk219 commented on issue #12745:
URL: https://github.com/apache/pinot/issues/12745#issuecomment-2029142242
Sure, I will take this up.
--
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 sp
jackjlli commented on issue #12745:
URL: https://github.com/apache/pinot/issues/12745#issuecomment-2026438959
Thanks @jtao15 for reporting! Would @shounakmk219 take this up? Also, does
it make more sense to unify the way of passing parameters for queries? There
are already quite a few ways
jtao15 commented on issue #12745:
URL: https://github.com/apache/pinot/issues/12745#issuecomment-2026428706
cc @vvivekiyer @Jackie-Jiang @jackjlli @siddharthteotia
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
U
