Jackie-Jiang commented on PR #12244: URL: https://github.com/apache/pinot/pull/12244#issuecomment-1958288910
Thanks for reporting the issue! I'm trying to figure out what could cause this max length mismatch. @eaugene Can you please provide the Pinot version you are running? In `FilterPlanNode`, `numDocs` is collected first (line 99 in current master), then operator is constructed, where `DataSource` is created (line 250 in current master), at which point `maxNumValuesPerMVEntry` is collected. In the consuming segment (`MutableSegmentImpl`), we update `maxNumValuesPerMVEntry` before `numDocsIndexed`, and we shouldn't run into this ideally unless I missed something. -- 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-mail: commits-unsubscr...@pinot.apache.org 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
