You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@bigtop.apache.org by GitBox <gi...@apache.org> on 2021/02/03 07:09:08 UTC

[GitHub] [bigtop] iwasakims opened a new pull request #722: BIGTOP-3486. Fix version mismatch of Jetty between Hadoop and HBase.

iwasakims opened a new pull request #722:
URL: https://github.com/apache/bigtop/pull/722


   https://issues.apache.org/jira/browse/BIGTOP-3486
   
   This sets Jetty version to the latest of 9.3.
   


----------------------------------------------------------------
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] [bigtop] sekikn commented on pull request #722: BIGTOP-3486. Fix version mismatch of Jetty between Hadoop and HBase.

Posted by GitBox <gi...@apache.org>.
sekikn commented on pull request #722:
URL: https://github.com/apache/bigtop/pull/722#issuecomment-773005522


   +1, building HBase with this PR succeeded on CentOS 7, Debian 10 and Ubuntu 18.04. Its smoke test also passed on CentOS 7. Thanks @iwasakims!


----------------------------------------------------------------
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] [bigtop] sekikn merged pull request #722: BIGTOP-3486. Fix version mismatch of Jetty between Hadoop and HBase.

Posted by GitBox <gi...@apache.org>.
sekikn merged pull request #722:
URL: https://github.com/apache/bigtop/pull/722


   


----------------------------------------------------------------
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] [bigtop] iwasakims commented on pull request #722: BIGTOP-3486. Fix version mismatch of Jetty between Hadoop and HBase.

Posted by GitBox <gi...@apache.org>.
iwasakims commented on pull request #722:
URL: https://github.com/apache/bigtop/pull/722#issuecomment-772292631


   Unit tests of Hadoop and HBase worked as expected with the patch applied.
   
   hadoop::
   ```
   mvn test -Dzookeeper.version=3.4.14 -Djetty.version=9.3.29.v20201019 -Dmaven.test.failure.ignore=true 
   ```
   
   hbase::
   ```
   mvn test -Phadoop-3.0 -Dhadoop-three.version=3.2.2 -Dslf4j.version=1.7.25 -Djetty.version=9.3.29.v20201019 -Dzookeeper.version=3.4.14 -Dsurefire.testFailureIgnore=true
   ```
   


----------------------------------------------------------------
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] [bigtop] sekikn commented on pull request #722: BIGTOP-3486. Fix version mismatch of Jetty between Hadoop and HBase.

Posted by GitBox <gi...@apache.org>.
sekikn commented on pull request #722:
URL: https://github.com/apache/bigtop/pull/722#issuecomment-772918194


   @iwasakims I tried to test this PR locally but it couldn't be applied as it is after merging #723. Would you rebase this?


----------------------------------------------------------------
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] [bigtop] iwasakims commented on pull request #722: BIGTOP-3486. Fix version mismatch of Jetty between Hadoop and HBase.

Posted by GitBox <gi...@apache.org>.
iwasakims commented on pull request #722:
URL: https://github.com/apache/bigtop/pull/722#issuecomment-772948653


   I rebased this. Thanks, @sekikn.


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