jadami10 commented on PR #12731: URL: https://github.com/apache/pinot/pull/12731#issuecomment-2038585813
I'm happy to close this, especially since jackie is tackling it with https://github.com/apache/pinot/pull/12789. I guess it's a matter of do we consider the pruners as optimizations or essential to the query? It seems your approaches are that a query is not just the data it returns but also its performance, and substantially affecting the performance is cause to fail the query. If this were to happen again, we could just as easily/quickly remove the pruner to mitigate impact. Whereas if we were to swallow all exceptions and run the query anyway, there would be no way for an operator to deal with now suboptimal queries. Both of your points make sense! -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
