cbalci opened a new pull request, #9837: URL: https://github.com/apache/pinot/pull/9837
Fixes a bug where an exception is thrown when all of the data tables retuned in a Spark 'split' is empty. When the Spark reader pushes down a filter which effectively reduces the returned data for all the segments to zero, GRPC data fetcher (`PinotGrpcServerDataFetcher`) wrongly assumes that there was an error. This behavior was introduced when porting logic from the non-GRPC fetcher (`PinotServerDataFetcher`) where the assumption is correct. Http and GRPC interfaces differ in behavior where the HTTP interface will return empty instances of datatables whereas with GRPC you only get metadata. I'm updating the error handling to remove this assumption and record an error when there is actual exception thrown by the GRPC server. `bugfix` -- 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
