anew commented on code in PR #57722:
URL: https://github.com/apache/spark/pull/57722#discussion_r3723821211
##########
sql/pipelines/src/main/scala/org/apache/spark/sql/pipelines/graph/DataflowGraph.scala:
##########
@@ -259,10 +271,12 @@ case class DataflowGraph(
def resolved: Boolean =
flows.forall(f => resolvedFlow.contains(f.identifier))
- def resolve(): DataflowGraph =
+ def resolve(sessionCaseSensitive: Boolean): DataflowGraph =
Review Comment:
thanks, I should have waited for the CI before saying its ready for review
(codex had claimed that it compiled and ran the tests... apparently not all of
them).
I decided to add the explicit parameters everywhere.
--
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]
