Jackie-Jiang commented on pull request #6139:
URL: https://github.com/apache/incubator-pinot/pull/6139#issuecomment-713044724
@jackjlli I see the problem, basically `tyrus` copied the code from
`grizzly` and put it in one package. If you upgrade `tyrus` to `1.15` to be
compatible with
Jackie-Jiang commented on pull request #6139:
URL: https://github.com/apache/incubator-pinot/pull/6139#issuecomment-712456684
> We cannot exclude either one of them because `tyrus` is needed to stream
external data and `jersey-container-grizzly2-http` is needed to start the web
server.
Jackie-Jiang commented on pull request #6139:
URL: https://github.com/apache/incubator-pinot/pull/6139#issuecomment-710553719
Isn't this just a version conflict? Does excluding this conflicting library
from one of the library work?
Jackie-Jiang commented on pull request #6139:
URL: https://github.com/apache/incubator-pinot/pull/6139#issuecomment-710493714
@mayankshriv @jackjlli I'm okay using static file for testing purpose only,
but if we remove `tyrus`, does that mean we can no longer consume real
streaming events
Jackie-Jiang commented on pull request #6139:
URL: https://github.com/apache/incubator-pinot/pull/6139#issuecomment-709638616
I'm against changing the data to a static file. The purpose of the quick
start is letting the user play with Pinot and they should be able to see the
latest events
