srowen commented on a change in pull request #26636: [SPARK-27721][BUILD]
Switch to use right leveldbjni according to the platforms
URL: https://github.com/apache/spark/pull/26636#discussion_r350048409
##########
File path: pom.xml
##########
@@ -3081,5 +3094,17 @@
<profile>
<id>sparkr</id>
</profile>
+ <profile>
+ <id>aarch64</id>
+ <properties>
+ <leveldbjni.group>org.openlabtesting.leveldbjni</leveldbjni.group>
+ </properties>
+ <activation>
+ <os>
+ <family>linux</family>
+ <arch>aarch64</arch>
Review comment:
The most ideal outcome would be for the upstream project to adopt the
change. It looks quite minor. Otherwise the fork might end up running behind
upstream changes, but it sounds like there is not much upstream activity.
Next-most ideal would be for Hadoop to adopt the same approach and we use a
release that follows the same pattern. But the approach here is OK and maybe
less intrusive as it won't affect the main build.
----------------------------------------------------------------
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]
