c21 edited a comment on pull request #31708: URL: https://github.com/apache/spark/pull/31708#issuecomment-789474307
@cloud-fan - thanks for review. > Can you help to improve the test framework to include the number of shuffles, reused shuffles, subqueries, and reused subqueries in the golden files? Sure. I agree this it, and I also found plan check during test failure is quite hard for complicated query. I am actually thinking to add information for ALL physical operators. The added content like: ``` Total number of operators: 6 HashAggregate: 3 Exchange: 2 Scan: 1 ``` WDYT? ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
