New Query Runner: Apache Pinot

[justrp]

What type of PR is this? (check all applicable)

• New Query Runner (Data Source)

Description

Query runner for http://pinot.apache.org/

Related Tickets & Documents

apache/pinot#6297

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

[etadelta222]

Hi @kravets-levko , has this been abandoned?

[justrp]

oh, wow, I almost forgot about this one: it's been almost a year)
rebased from current master branch

[etadelta222]

Hi @arikfr , @rauchy , can someone help with this?

[susodapop]

Hey @justrp and @etadelta222 thanks so much for the ping. We just introduced our new process for reviews which includes a new PR template you can see here. Would you mind updating the PR description? Really all we need to see is the applicable versions of pinot for which this query runner will work.

I'm looking forward to getting this reviewed and merged ASAP!

[xiangfu0]

Can you add more types here, especially the TIMESTAMP

[xiangfu0]

0.3.11 is the latest version :p

[etadelta222]

Hi @susodapop , is the information provided by Xiangfu sufficient?

[justrp]

in case if you wait for me to address these changes - unfortunately, can't provide clear ETA - got a very busy time now :(
hopefully, will have some time at week after the next one - but can't predict for sure

[susodapop]

Another contributor has opened #5724 which duplicates this PR. Since @justrp is currently busy, I'm closing this one for now but have asked the author of 5724 to incorporate @xiangfu0's feedback into their effort.