steveburnett commented on code in PR #10793: URL: https://github.com/apache/incubator-gluten/pull/10793#discussion_r2379455784
########## docs/developers/NewToGluten.md: ########## @@ -498,20 +439,20 @@ spark-shell --name run_gluten \ --conf spark.shuffle.manager=org.apache.spark.shuffle.sort.ColumnarShuffleManager ``` -# Check Gluten Approved Spark Plan +## Check Gluten Approved Spark Plan To make sure we don't accidentally modify the Gluten and Spark Plan build logic. We introduce new logic in `VeloxTPCHSuite` to check whether the plan has been changed or not, and this will be triggered when running the unit test. Review Comment: Your update makes sense to me! I have no suggestions to make to these latest changes. -- 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]
