mayankshriv commented on pull request #5872: URL: https://github.com/apache/incubator-pinot/pull/5872#issuecomment-675781224
> @mayankshriv Can you please check the existing use cases on `distinctCount`? If the column is not STRING type, then the overhead should be minimal. > For the existing behavior, you may use `distinctCountBitmap` which is the enhanced version of the current `distinctCount` using the `RoaringBitmap` @Jackie-Jiang We have thousands of tables at LinkedIn, so they will likely cover all kinds of distinctCount use cases. I don't quite follow your suggestion on us using `distinctCountBitmap`. Are you suggesting we ask our customers to migrate to the bitmap based implementation? ---------------------------------------------------------------- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
