LuciferYang commented on a change in pull request #30784:
URL: https://github.com/apache/spark/pull/30784#discussion_r544774883
##########
File path: pom.xml
##########
@@ -3220,9 +3217,33 @@
<profile>
<id>scala-2.12</id>
+ <activation>
Review comment:
> Generally the "2.12" configuration goes in the main POM and then is
overridden for non-default profiles here. So if 2.13 needs different config,
that's fine, but 2.12's config should be up top. If the problem is overwriting
Maven config in a profile, I can show you how to do that.
I'll try to solve it first
----------------------------------------------------------------
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]
