You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2020/11/17 14:34:46 UTC
[GitHub] [accumulo] jmark99 opened a new pull request #1785: Update hadoop version to 3.3.0 in pom.xml
jmark99 opened a new pull request #1785:
URL: https://github.com/apache/accumulo/pull/1785
Update version of hadoop to 3.3.0 in order to remove warnings. With 3.3.0 version the "An illegal reflective access operaion has occurred" errors no longer appear as the issue has been corrected in that version of Hadoop (actually, corrected in 3.2.2 and above).
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [accumulo] ctubbsii commented on a change in pull request #1785: Update hadoop version to 3.3.0 in pom.xml
Posted by GitBox <gi...@apache.org>.
ctubbsii commented on a change in pull request #1785:
URL: https://github.com/apache/accumulo/pull/1785#discussion_r525221464
##########
File path: pom.xml
##########
@@ -124,7 +124,7 @@
<failsafe.excludedGroups />
<failsafe.forkCount>1</failsafe.forkCount>
<failsafe.groups />
- <hadoop.version>3.2.1</hadoop.version>
+ <hadoop.version>3.3.0</hadoop.version>
Review comment:
I'd be okay with bumping to 3.2.2, because I think it's reasonable to support the latest bugfix in the earlier Hadoop release line. If we bump to 3.3.0, we may risk making changes that work for 3.3.x, but will cause a failure with 3.2.x. However, that may also be okay, since we have a build test to ensure it builds with versions as old as 3.0.3. What do you think?
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [accumulo] jmark99 merged pull request #1785: Update hadoop version to 3.3.0 in pom.xml
Posted by GitBox <gi...@apache.org>.
jmark99 merged pull request #1785:
URL: https://github.com/apache/accumulo/pull/1785
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [accumulo] jmark99 commented on a change in pull request #1785: Update hadoop version to 3.3.0 in pom.xml
Posted by GitBox <gi...@apache.org>.
jmark99 commented on a change in pull request #1785:
URL: https://github.com/apache/accumulo/pull/1785#discussion_r525225176
##########
File path: pom.xml
##########
@@ -124,7 +124,7 @@
<failsafe.excludedGroups />
<failsafe.forkCount>1</failsafe.forkCount>
<failsafe.groups />
- <hadoop.version>3.2.1</hadoop.version>
+ <hadoop.version>3.3.0</hadoop.version>
Review comment:
We can go with 3.2.2 just to take a more cautious approach. No need to chance breaking something if not necessary. I'll make the change.
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [accumulo] ctubbsii commented on a change in pull request #1785: Update hadoop version to 3.3.0 in pom.xml
Posted by GitBox <gi...@apache.org>.
ctubbsii commented on a change in pull request #1785:
URL: https://github.com/apache/accumulo/pull/1785#discussion_r525228593
##########
File path: pom.xml
##########
@@ -124,7 +124,7 @@
<failsafe.excludedGroups />
<failsafe.forkCount>1</failsafe.forkCount>
<failsafe.groups />
- <hadoop.version>3.2.1</hadoop.version>
+ <hadoop.version>3.3.0</hadoop.version>
Review comment:
I don't even see 3.2.2 listed on their website. Maybe it would be best to stick with 3.3.0 as the default.
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [accumulo] jmark99 commented on a change in pull request #1785: Update hadoop version to 3.3.0 in pom.xml
Posted by GitBox <gi...@apache.org>.
jmark99 commented on a change in pull request #1785:
URL: https://github.com/apache/accumulo/pull/1785#discussion_r525229381
##########
File path: pom.xml
##########
@@ -124,7 +124,7 @@
<failsafe.excludedGroups />
<failsafe.forkCount>1</failsafe.forkCount>
<failsafe.groups />
- <hadoop.version>3.2.1</hadoop.version>
+ <hadoop.version>3.3.0</hadoop.version>
Review comment:
Interesting! I'll revert back to 3.3.0 :)
----------------------------------------------------------------
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:
users@infra.apache.org