huonw commented on issue #24414: [SPARK-22044][SQL] Add `cost` and `codegen` arguments to `explain` URL: https://github.com/apache/spark/pull/24414#issuecomment-486881045 I think that `test hint` test failure is "unrelated": it is caused by https://github.com/apache/spark/commit/ad60c6d9be3234a0296d1620129d5ca108f0876b breaking `explain(extended = true)` (see https://issues.apache.org/jira/browse/SPARK-27439?focusedCommentId=16826542&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-16826542), which this PR happens to be affected by because it makes the R `explain` (which a test was relying on) have the same incorrect output as the Scala one. I'll change the test to extract the relevant info directly. (Also, I think none of those public classes are added by this PR: it seems like they're already in `master` and being picked up because of the merge commit 2cb0489?)
---------------------------------------------------------------- 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] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
