You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2020/12/14 21:06:38 UTC

[GitHub] [incubator-pinot] mayankshriv removed a comment on pull request #5926: Introduce IdSet and add IdSetAggregationFunction

mayankshriv removed a comment on pull request #5926:
URL: https://github.com/apache/incubator-pinot/pull/5926#issuecomment-744633367


   @Jackie-Jiang Any chance you can break the PR to have the thrift side changes separated from actual code changes? Or do they go hand in hand? Might be easier to review the PR if broken.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org