AnishMahto commented on code in PR #50875:
URL: https://github.com/apache/spark/pull/50875#discussion_r2096432380
##########
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/UnsupportedOperationsSuite.scala:
##########
@@ -1063,7 +1063,7 @@ class UnsupportedOperationsSuite extends SparkFunSuite
with SQLHelper {
/** Assert that the logical plan is supported as a batch plan */
def assertSupportedInBatchPlan(name: String, plan: LogicalPlan): Unit = {
test(s"batch plan - $name: supported") {
- UnsupportedOperationChecker.checkForBatch(plan)
+ UnsupportedOperationChecker.checkForBatch(plan = plan)
Review Comment:
Ah yeah this is an unneeded artifact of a reverted change (within this
branch). Previously I was also plumbing through an `isSql` boolean argument
which I needed to name, so I figured I might as well name both arguments.
Later I decided to remove that second argument, but forgot to de-name the
plan argument too. Done
--
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: reviews-unsubscr...@spark.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
