You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by GitBox <gi...@apache.org> on 2020/05/27 04:16:26 UTC

[GitHub] [hbase-connectors] CodingBen opened a new pull request #68: HBASE-20999 Move hbase-REST to new hbase-connectors repository

CodingBen opened a new pull request #68:
URL: https://github.com/apache/hbase-connectors/pull/68


   


----------------------------------------------------------------
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] [hbase-connectors] Apache-HBase commented on pull request #68: HBASE-20999 Move hbase-REST to new hbase-connectors repository

Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on pull request #68:
URL: https://github.com/apache/hbase-connectors/pull/68#issuecomment-920552964


   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |:----:|----------:|--------:|:--------|
   | +0 :ok: |  reexec  |   0m 40s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  0s |  No case conflicting files found.  |
   | +0 :ok: |  prototool  |   0m  0s |  prototool was not available.  |
   | +0 :ok: |  spotbugs  |   0m  0s |  spotbugs executables are not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  The patch does not contain any @author tags.  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  The patch appears to include 49 new or modified test files.  |
   ||| _ master Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   6m 44s |  master passed  |
   | +1 :green_heart: |  compile  |   0m 49s |  master passed  |
   | +1 :green_heart: |  javadoc  |   0m 50s |  master passed  |
   | -0 :warning: |  patch  |   9m 14s |  Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 15s |  Maven dependency ordering for patch  |
   | -1 :x: |  mvninstall  |   0m 51s |  root in the patch failed.  |
   | -1 :x: |  compile  |   0m 53s |  root in the patch failed.  |
   | -1 :x: |  cc  |   0m 53s |  root in the patch failed.  |
   | -1 :x: |  javac  |   0m 53s |  root in the patch failed.  |
   | -1 :x: |  whitespace  |   0m  0s |  The patch has 61 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply  |
   | -1 :x: |  whitespace  |   0m  0s |  The patch 1 line(s) with tabs.  |
   | +1 :green_heart: |  xml  |   0m  6s |  The patch has no ill-formed XML file.  |
   | -1 :x: |  javadoc  |   0m 51s |  root in the patch failed.  |
   ||| _ Other Tests _ |
   | -1 :x: |  unit  |   7m 23s |  root in the patch failed.  |
   |  |   |  20m 33s |   |
   
   
   | Subsystem | Report/Notes |
   |----------:|:-------------|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/Dockerfile |
   | GITHUB PR | https://github.com/apache/hbase-connectors/pull/68 |
   | Optional Tests | dupname javac javadoc unit xml compile cc prototool spotbugs findbugs |
   | uname | Linux 9714c6b1e629 5.4.0-1025-aws #25~18.04.1-Ubuntu SMP Fri Sep 11 12:03:04 UTC 2020 x86_64 GNU/Linux |
   | Build tool | hb_maven |
   | Personality | dev-support/jenkins/hbase-personality.sh |
   | git revision | master / 37aa8d5 |
   | Default Java | Oracle Corporation-1.8.0_282-b08 |
   | mvninstall | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/patch-mvninstall-root.txt |
   | compile | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/patch-compile-root.txt |
   | cc | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/patch-compile-root.txt |
   | javac | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/patch-compile-root.txt |
   | whitespace | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/whitespace-eol.txt |
   | whitespace | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/whitespace-tabs.txt |
   | javadoc | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/patch-javadoc-root.txt |
   | unit | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/patch-unit-root.txt |
   |  Test Results | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/testReport/ |
   | Max. process+thread count | 918 (vs. ulimit of 12500) |
   | modules | C: rest rest/hbase-rest-protocol . rest/hbase-rest U: . |
   | Console output | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/console |
   | versions | git=2.20.1 |
   | Powered by | Apache Yetus 0.12.0 https://yetus.apache.org |
   
   
   This message was automatically generated.
   
   


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

To unsubscribe, e-mail: issues-unsubscribe@hbase.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [hbase-connectors] Apache-HBase commented on pull request #68: HBASE-20999 Move hbase-REST to new hbase-connectors repository

Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on pull request #68:
URL: https://github.com/apache/hbase-connectors/pull/68#issuecomment-924550613


   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |:----:|----------:|--------:|:--------|
   | +0 :ok: |  reexec  |   1m  3s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  1s |  No case conflicting files found.  |
   | +0 :ok: |  prototool  |   0m  0s |  prototool was not available.  |
   | +0 :ok: |  spotbugs  |   0m  0s |  spotbugs executables are not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  The patch does not contain any @author tags.  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  The patch appears to include 49 new or modified test files.  |
   ||| _ master Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   1m 44s |  master passed  |
   | +1 :green_heart: |  compile  |   0m 49s |  master passed  |
   | +1 :green_heart: |  javadoc  |   0m 49s |  master passed  |
   | -0 :warning: |  patch  |   4m 13s |  Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 15s |  Maven dependency ordering for patch  |
   | -1 :x: |  mvninstall  |   0m 51s |  root in the patch failed.  |
   | -1 :x: |  compile  |   0m 56s |  root in the patch failed.  |
   | -1 :x: |  cc  |   0m 56s |  root in the patch failed.  |
   | -1 :x: |  javac  |   0m 56s |  root in the patch failed.  |
   | -1 :x: |  whitespace  |   0m  1s |  The patch has 61 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply  |
   | -1 :x: |  whitespace  |   0m  1s |  The patch 1 line(s) with tabs.  |
   | +1 :green_heart: |  xml  |   0m  5s |  The patch has no ill-formed XML file.  |
   | -1 :x: |  javadoc  |   0m 50s |  root in the patch failed.  |
   ||| _ Other Tests _ |
   | -1 :x: |  unit  |   7m 23s |  root in the patch failed.  |
   |  |   |  15m 51s |   |
   
   
   | Subsystem | Report/Notes |
   |----------:|:-------------|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/Dockerfile |
   | GITHUB PR | https://github.com/apache/hbase-connectors/pull/68 |
   | Optional Tests | dupname javac javadoc unit xml compile cc prototool spotbugs findbugs |
   | uname | Linux 154b5f62fbae 5.4.0-1025-aws #25~18.04.1-Ubuntu SMP Fri Sep 11 12:03:04 UTC 2020 x86_64 GNU/Linux |
   | Build tool | hb_maven |
   | Personality | dev-support/jenkins/hbase-personality.sh |
   | git revision | master / 37aa8d5 |
   | Default Java | Oracle Corporation-1.8.0_282-b08 |
   | mvninstall | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/patch-mvninstall-root.txt |
   | compile | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/patch-compile-root.txt |
   | cc | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/patch-compile-root.txt |
   | javac | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/patch-compile-root.txt |
   | whitespace | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/whitespace-eol.txt |
   | whitespace | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/whitespace-tabs.txt |
   | javadoc | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/patch-javadoc-root.txt |
   | unit | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/patch-unit-root.txt |
   |  Test Results | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/testReport/ |
   | Max. process+thread count | 918 (vs. ulimit of 12500) |
   | modules | C: rest rest/hbase-rest-protocol . rest/hbase-rest U: . |
   | Console output | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/console |
   | versions | git=2.20.1 |
   | Powered by | Apache Yetus 0.12.0 https://yetus.apache.org |
   
   
   This message was automatically generated.
   
   


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

To unsubscribe, e-mail: issues-unsubscribe@hbase.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [hbase-connectors] Apache-HBase commented on pull request #68: HBASE-20999 Move hbase-REST to new hbase-connectors repository

Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on pull request #68:
URL: https://github.com/apache/hbase-connectors/pull/68#issuecomment-683755937


   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |:----:|----------:|--------:|:--------|
   | +0 :ok: |  reexec  |   1m 19s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  1s |  No case conflicting files found.  |
   | +0 :ok: |  prototool  |   0m  0s |  prototool was not available.  |
   | +0 :ok: |  spotbugs  |   0m  0s |  spotbugs executables are not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  The patch does not contain any @author tags.  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  The patch appears to include 49 new or modified test files.  |
   ||| _ master Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   2m  9s |  master passed  |
   | +1 :green_heart: |  compile  |   1m 34s |  master passed  |
   | +1 :green_heart: |  javadoc  |   1m 34s |  master passed  |
   | -0 :warning: |  patch  |   6m 53s |  Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 17s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   1m 53s |  the patch passed  |
   | +1 :green_heart: |  compile  |   1m 57s |  the patch passed  |
   | +1 :green_heart: |  cc  |   1m 57s |  the patch passed  |
   | +1 :green_heart: |  javac  |   1m 57s |  the patch passed  |
   | -1 :x: |  whitespace  |   0m  0s |  The patch has 61 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply  |
   | -1 :x: |  whitespace  |   0m  0s |  The patch 1 line(s) with tabs.  |
   | +1 :green_heart: |  xml  |   0m 10s |  The patch has no ill-formed XML file.  |
   | +1 :green_heart: |  javadoc  |   1m 48s |  the patch passed  |
   ||| _ Other Tests _ |
   | -1 :x: |  unit  |  25m 22s |  root in the patch failed.  |
   |  |   |  40m  5s |   |
   
   
   | Reason | Tests |
   |-------:|:------|
   | Failed junit tests | hadoop.hbase.rest.TestGzipFilter |
   |   | hadoop.hbase.rest.TestMultiRowResource |
   |   | hadoop.hbase.rest.TestTableScan |
   |   | hadoop.hbase.rest.TestSecureRESTServer |
   |   | hadoop.hbase.rest.TestVersionResource |
   |   | hadoop.hbase.rest.TestSecurityHeadersFilter |
   |   | hadoop.hbase.rest.TestNamespacesInstanceResource |
   |   | hadoop.hbase.rest.TestSchemaResource |
   |   | hadoop.hbase.rest.TestTableResource |
   |   | hadoop.hbase.rest.TestScannersWithFilters |
   |   | hadoop.hbase.rest.TestResourceFilter |
   |   | hadoop.hbase.rest.TestStatusResource |
   |   | hadoop.hbase.rest.TestScannerResource |
   |   | hadoop.hbase.rest.TestDeleteRow |
   |   | hadoop.hbase.rest.client.TestRemoteTable |
   |   | hadoop.hbase.rest.TestNamespacesResource |
   |   | hadoop.hbase.rest.TestGetAndPutResource |
   |   | hadoop.hbase.rest.TestScannersWithLabels |
   
   
   | Subsystem | Report/Notes |
   |----------:|:-------------|
   | Docker | ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/4/artifact/yetus-precommit-check/output/Dockerfile |
   | GITHUB PR | https://github.com/apache/hbase-connectors/pull/68 |
   | Optional Tests | dupname javac javadoc unit xml compile cc prototool spotbugs findbugs |
   | uname | Linux a69e71fe3893 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 GNU/Linux |
   | Build tool | hb_maven |
   | Personality | dev-support/jenkins/hbase-personality.sh |
   | git revision | master / d3175ad |
   | Default Java | Oracle Corporation-1.8.0_265-b01 |
   | whitespace | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/4/artifact/yetus-precommit-check/output/whitespace-eol.txt |
   | whitespace | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/4/artifact/yetus-precommit-check/output/whitespace-tabs.txt |
   | unit | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/4/artifact/yetus-precommit-check/output/patch-unit-root.txt |
   |  Test Results | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/4/testReport/ |
   | Max. process+thread count | 958 (vs. ulimit of 12500) |
   | modules | C: rest rest/hbase-rest-protocol . rest/hbase-rest U: . |
   | Console output | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/4/console |
   | versions | git=2.20.1 |
   | Powered by | Apache Yetus 0.12.0 https://yetus.apache.org |
   
   
   This message was automatically generated.
   
   


----------------------------------------------------------------
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] [hbase-connectors] Apache-HBase commented on pull request #68: HBASE-20999 Move hbase-REST to new hbase-connectors repository

Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on pull request #68:
URL: https://github.com/apache/hbase-connectors/pull/68#issuecomment-827412238


   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |:----:|----------:|--------:|:--------|
   | +0 :ok: |  reexec  |   1m 42s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  1s |  No case conflicting files found.  |
   | +0 :ok: |  prototool  |   0m  0s |  prototool was not available.  |
   | +0 :ok: |  spotbugs  |   0m  0s |  spotbugs executables are not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  The patch does not contain any @author tags.  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  The patch appears to include 49 new or modified test files.  |
   ||| _ master Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   1m 59s |  master passed  |
   | +1 :green_heart: |  compile  |   0m 58s |  master passed  |
   | +1 :green_heart: |  javadoc  |   0m 52s |  master passed  |
   | -0 :warning: |  patch  |   4m 50s |  Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 16s |  Maven dependency ordering for patch  |
   | -1 :x: |  mvninstall  |   1m  0s |  root in the patch failed.  |
   | -1 :x: |  compile  |   1m  0s |  root in the patch failed.  |
   | -1 :x: |  cc  |   1m  0s |  root in the patch failed.  |
   | -1 :x: |  javac  |   1m  0s |  root in the patch failed.  |
   | -1 :x: |  whitespace  |   0m  0s |  The patch has 61 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply  |
   | -1 :x: |  whitespace  |   0m  0s |  The patch 1 line(s) with tabs.  |
   | +1 :green_heart: |  xml  |   0m  5s |  The patch has no ill-formed XML file.  |
   | -1 :x: |  javadoc  |   0m 49s |  root in the patch failed.  |
   ||| _ Other Tests _ |
   | -1 :x: |  unit  |   7m 31s |  root in the patch failed.  |
   |  |   |  17m 27s |   |
   
   
   | Subsystem | Report/Notes |
   |----------:|:-------------|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/Dockerfile |
   | GITHUB PR | https://github.com/apache/hbase-connectors/pull/68 |
   | Optional Tests | dupname javac javadoc unit xml compile cc prototool spotbugs findbugs |
   | uname | Linux 1d9c225643b7 5.4.0-1025-aws #25~18.04.1-Ubuntu SMP Fri Sep 11 12:03:04 UTC 2020 x86_64 GNU/Linux |
   | Build tool | hb_maven |
   | Personality | dev-support/jenkins/hbase-personality.sh |
   | git revision | master / fddb433 |
   | Default Java | Oracle Corporation-1.8.0_282-b08 |
   | mvninstall | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/patch-mvninstall-root.txt |
   | compile | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/patch-compile-root.txt |
   | cc | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/patch-compile-root.txt |
   | javac | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/patch-compile-root.txt |
   | whitespace | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/whitespace-eol.txt |
   | whitespace | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/whitespace-tabs.txt |
   | javadoc | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/patch-javadoc-root.txt |
   | unit | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/artifact/yetus-precommit-check/output/patch-unit-root.txt |
   |  Test Results | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/testReport/ |
   | Max. process+thread count | 916 (vs. ulimit of 12500) |
   | modules | C: rest rest/hbase-rest-protocol . rest/hbase-rest U: . |
   | Console output | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/1/console |
   | versions | git=2.20.1 |
   | Powered by | Apache Yetus 0.12.0 https://yetus.apache.org |
   
   
   This message was automatically generated.
   
   


-- 
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] [hbase-connectors] infraio commented on pull request #68: HBASE-20999 Move hbase-REST to new hbase-connectors repository

Posted by GitBox <gi...@apache.org>.
infraio commented on pull request #68:
URL: https://github.com/apache/hbase-connectors/pull/68#issuecomment-634467845


   > I think the idea is to move the hbase-rest module out to https://github.com/apache/hbase-connectors where it'd live instead as a REST submodule.
   
   A new repo?


----------------------------------------------------------------
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] [hbase-connectors] Apache-HBase commented on pull request #68: HBASE-20999 Move hbase-REST to new hbase-connectors repository

Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on pull request #68:
URL: https://github.com/apache/hbase-connectors/pull/68#issuecomment-683777278


   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |:----:|----------:|--------:|:--------|
   | +0 :ok: |  reexec  |   1m 16s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  2s |  No case conflicting files found.  |
   | +0 :ok: |  prototool  |   0m  0s |  prototool was not available.  |
   | +0 :ok: |  spotbugs  |   0m  0s |  spotbugs executables are not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  The patch does not contain any @author tags.  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  The patch appears to include 49 new or modified test files.  |
   ||| _ master Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   2m 13s |  master passed  |
   | +1 :green_heart: |  compile  |   1m 36s |  master passed  |
   | +1 :green_heart: |  javadoc  |   1m 43s |  master passed  |
   | -0 :warning: |  patch  |   7m 11s |  Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 17s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   1m 45s |  the patch passed  |
   | +1 :green_heart: |  compile  |   1m 57s |  the patch passed  |
   | +1 :green_heart: |  cc  |   1m 57s |  the patch passed  |
   | +1 :green_heart: |  javac  |   1m 57s |  the patch passed  |
   | -1 :x: |  whitespace  |   0m  1s |  The patch has 61 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply  |
   | -1 :x: |  whitespace  |   0m  1s |  The patch 1 line(s) with tabs.  |
   | +1 :green_heart: |  xml  |   0m 11s |  The patch has no ill-formed XML file.  |
   | +1 :green_heart: |  javadoc  |   1m 48s |  the patch passed  |
   ||| _ Other Tests _ |
   | -1 :x: |  unit  |  25m 20s |  root in the patch failed.  |
   |  |   |  40m 11s |   |
   
   
   | Reason | Tests |
   |-------:|:------|
   | Failed junit tests | hadoop.hbase.rest.TestGzipFilter |
   |   | hadoop.hbase.rest.TestMultiRowResource |
   |   | hadoop.hbase.rest.TestTableScan |
   |   | hadoop.hbase.rest.TestSecureRESTServer |
   |   | hadoop.hbase.rest.TestVersionResource |
   |   | hadoop.hbase.rest.TestSecurityHeadersFilter |
   |   | hadoop.hbase.rest.TestNamespacesInstanceResource |
   |   | hadoop.hbase.rest.TestSchemaResource |
   |   | hadoop.hbase.rest.TestTableResource |
   |   | hadoop.hbase.rest.TestScannersWithFilters |
   |   | hadoop.hbase.rest.TestResourceFilter |
   |   | hadoop.hbase.rest.TestStatusResource |
   |   | hadoop.hbase.rest.TestScannerResource |
   |   | hadoop.hbase.rest.TestDeleteRow |
   |   | hadoop.hbase.rest.client.TestRemoteTable |
   |   | hadoop.hbase.rest.TestNamespacesResource |
   |   | hadoop.hbase.rest.TestGetAndPutResource |
   |   | hadoop.hbase.rest.TestScannersWithLabels |
   
   
   | Subsystem | Report/Notes |
   |----------:|:-------------|
   | Docker | ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/5/artifact/yetus-precommit-check/output/Dockerfile |
   | GITHUB PR | https://github.com/apache/hbase-connectors/pull/68 |
   | Optional Tests | dupname javac javadoc unit xml compile cc prototool spotbugs findbugs |
   | uname | Linux d08ed9668fbc 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 GNU/Linux |
   | Build tool | hb_maven |
   | Personality | dev-support/jenkins/hbase-personality.sh |
   | git revision | master / d3175ad |
   | Default Java | Oracle Corporation-1.8.0_265-b01 |
   | whitespace | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/5/artifact/yetus-precommit-check/output/whitespace-eol.txt |
   | whitespace | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/5/artifact/yetus-precommit-check/output/whitespace-tabs.txt |
   | unit | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/5/artifact/yetus-precommit-check/output/patch-unit-root.txt |
   |  Test Results | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/5/testReport/ |
   | Max. process+thread count | 955 (vs. ulimit of 12500) |
   | modules | C: rest rest/hbase-rest-protocol . rest/hbase-rest U: . |
   | Console output | https://ci-hadoop.apache.org/job/HBase/job/HBase-Connectors-PreCommit/job/PR-68/5/console |
   | versions | git=2.20.1 |
   | Powered by | Apache Yetus 0.12.0 https://yetus.apache.org |
   
   
   This message was automatically generated.
   
   


----------------------------------------------------------------
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] [hbase-connectors] saintstack commented on a change in pull request #68: HBASE-20999 Move hbase-REST to new hbase-connectors repository

Posted by GitBox <gi...@apache.org>.
saintstack commented on a change in pull request #68:
URL: https://github.com/apache/hbase-connectors/pull/68#discussion_r430850242



##########
File path: hbase-connectors-protocol-shaded/pom.xml
##########
@@ -0,0 +1,275 @@
+<?xml version="1.0" encoding="UTF-8"?>

Review comment:
       License is missing.




----------------------------------------------------------------
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] [hbase-connectors] Apache-HBase commented on pull request #68: HBASE-20999 Move hbase-REST to new hbase-connectors repository

Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on pull request #68:
URL: https://github.com/apache/hbase-connectors/pull/68#issuecomment-1031055525


   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |:----:|----------:|--------:|:--------|
   | +0 :ok: |  reexec  |   0m 58s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  1s |  No case conflicting files found.  |
   | +0 :ok: |  prototool  |   0m  0s |  prototool was not available.  |
   | +0 :ok: |  spotbugs  |   0m  0s |  spotbugs executables are not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  The patch does not contain any @author tags.  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  The patch appears to include 49 new or modified test files.  |
   ||| _ master Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   1m 24s |  master passed  |
   | +1 :green_heart: |  compile  |   0m 51s |  master passed  |
   | +1 :green_heart: |  javadoc  |   0m 51s |  master passed  |
   | -0 :warning: |  patch  |   3m 59s |  Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 11s |  Maven dependency ordering for patch  |
   | -1 :x: |  mvninstall  |   0m 51s |  root in the patch failed.  |
   | -1 :x: |  compile  |   0m 57s |  root in the patch failed.  |
   | -1 :x: |  cc  |   0m 57s |  root in the patch failed.  |
   | -1 :x: |  javac  |   0m 57s |  root in the patch failed.  |
   | -1 :x: |  whitespace  |   0m  0s |  The patch has 61 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply  |
   | -1 :x: |  whitespace  |   0m  0s |  The patch 1 line(s) with tabs.  |
   | +1 :green_heart: |  xml  |   0m  5s |  The patch has no ill-formed XML file.  |
   | -1 :x: |  javadoc  |   0m 53s |  root in the patch failed.  |
   ||| _ Other Tests _ |
   | -1 :x: |  unit  |   8m 14s |  root in the patch failed.  |
   |  |   |  16m 24s |   |
   
   
   | Subsystem | Report/Notes |
   |----------:|:-------------|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-Connectors-PreCommit/job/PR-68/2/artifact/yetus-precommit-check/output/Dockerfile |
   | GITHUB PR | https://github.com/apache/hbase-connectors/pull/68 |
   | Optional Tests | dupname javac javadoc unit xml compile cc prototool spotbugs findbugs |
   | uname | Linux 35f999d14251 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 GNU/Linux |
   | Build tool | hb_maven |
   | Personality | dev-support/jenkins/hbase-personality.sh |
   | git revision | master / 2bfc5f1 |
   | Default Java | Oracle Corporation-1.8.0_282-b08 |
   | mvninstall | https://ci-hbase.apache.org/job/HBase-Connectors-PreCommit/job/PR-68/2/artifact/yetus-precommit-check/output/patch-mvninstall-root.txt |
   | compile | https://ci-hbase.apache.org/job/HBase-Connectors-PreCommit/job/PR-68/2/artifact/yetus-precommit-check/output/patch-compile-root.txt |
   | cc | https://ci-hbase.apache.org/job/HBase-Connectors-PreCommit/job/PR-68/2/artifact/yetus-precommit-check/output/patch-compile-root.txt |
   | javac | https://ci-hbase.apache.org/job/HBase-Connectors-PreCommit/job/PR-68/2/artifact/yetus-precommit-check/output/patch-compile-root.txt |
   | whitespace | https://ci-hbase.apache.org/job/HBase-Connectors-PreCommit/job/PR-68/2/artifact/yetus-precommit-check/output/whitespace-eol.txt |
   | whitespace | https://ci-hbase.apache.org/job/HBase-Connectors-PreCommit/job/PR-68/2/artifact/yetus-precommit-check/output/whitespace-tabs.txt |
   | javadoc | https://ci-hbase.apache.org/job/HBase-Connectors-PreCommit/job/PR-68/2/artifact/yetus-precommit-check/output/patch-javadoc-root.txt |
   | unit | https://ci-hbase.apache.org/job/HBase-Connectors-PreCommit/job/PR-68/2/artifact/yetus-precommit-check/output/patch-unit-root.txt |
   |  Test Results | https://ci-hbase.apache.org/job/HBase-Connectors-PreCommit/job/PR-68/2/testReport/ |
   | Max. process+thread count | 916 (vs. ulimit of 12500) |
   | modules | C: rest rest/hbase-rest-protocol . rest/hbase-rest U: . |
   | Console output | https://ci-hbase.apache.org/job/HBase-Connectors-PreCommit/job/PR-68/2/console |
   | versions | git=2.20.1 |
   | Powered by | Apache Yetus 0.12.0 https://yetus.apache.org |
   
   
   This message was automatically generated.
   
   


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

To unsubscribe, e-mail: issues-unsubscribe@hbase.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org