LuciferYang commented on a change in pull request #32648:
URL: https://github.com/apache/spark/pull/32648#discussion_r638743793



##########
File path: pom.xml
##########
@@ -3387,6 +3387,8 @@
                   <arg>-Wconf:msg=Auto-application to \`\(\)\` is 
deprecated:s</arg>
                   <arg>-Wconf:msg=method with a single empty parameter list 
overrides method without any parameter list:s</arg>
                   <arg>-Wconf:msg=method without a parameter list overrides a 
method with a single empty one:s</arg>
+                  <!-- SPARK-35496 Suppress `The outer reference in this type 
test cannot be checked at run time` -->

Review comment:
       Just to try not to modify SQLQueryTestSuite and 
ThriftServerQueryTestSuite, it may be useful to upgrade Scala 2.13.7 later

##########
File path: pom.xml
##########
@@ -3387,6 +3387,8 @@
                   <arg>-Wconf:msg=Auto-application to \`\(\)\` is 
deprecated:s</arg>
                   <arg>-Wconf:msg=method with a single empty parameter list 
overrides method without any parameter list:s</arg>
                   <arg>-Wconf:msg=method without a parameter list overrides a 
method with a single empty one:s</arg>
+                  <!-- SPARK-35496 Suppress `The outer reference in this type 
test cannot be checked at run time` -->

Review comment:
       Just to try not to change SQLQueryTestSuite and 
ThriftServerQueryTestSuite, it may be useful to upgrade Scala 2.13.7 later




-- 
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]

Reply via email to