[ 
https://issues.apache.org/jira/browse/PHOENIX-7158?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17823455#comment-17823455
 ] 

ASF GitHub Bot commented on PHOENIX-7158:
-----------------------------------------

stoty commented on PR #125:
URL: 
https://github.com/apache/phoenix-connectors/pull/125#issuecomment-1978084192

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |:----:|----------:|--------:|:--------|
   | +0 :ok: |  reexec  |   6m 17s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  0s |  No case conflicting files 
found.  |
   | +1 :green_heart: |  @author  |   0m  0s |  The patch does not contain any 
@author tags.  |
   | -1 :x: |  test4tests  |   0m  0s |  The patch doesn't appear to include 
any new or modified tests. Please justify why no new tests are needed for this 
patch. Also please list what manual steps were performed to verify this patch.  
|
   ||| _ master Compile Tests _ |
   | -1 :x: |  mvninstall  |   0m 12s |  root in master failed.  |
   | -1 :x: |  compile  |   0m  6s |  root in master failed.  |
   | -1 :x: |  javadoc  |   0m  7s |  root in master failed.  |
   ||| _ Patch Compile Tests _ |
   | -1 :x: |  mvninstall  |   0m  6s |  root in the patch failed.  |
   | -1 :x: |  compile  |   0m  6s |  root in the patch failed.  |
   | -1 :x: |  javac  |   0m  6s |  root in the patch failed.  |
   | +1 :green_heart: |  whitespace  |   0m  0s |  The patch has no whitespace 
issues.  |
   | +1 :green_heart: |  xml  |   0m  2s |  The patch has no ill-formed XML 
file.  |
   | -1 :x: |  javadoc  |   0m  6s |  root in the patch failed.  |
   ||| _ Other Tests _ |
   | -1 :x: |  unit  |   0m  7s |  root in the patch failed.  |
   | +1 :green_heart: |  asflicense  |   0m 19s |  The patch does not generate 
ASF License warnings.  |
   |  |   |   7m 46s |   |
   
   
   | Subsystem | Report/Notes |
   |----------:|:-------------|
   | Docker | ClientAPI=1.44 ServerAPI=1.44 base: 
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-125/4/artifact/yetus-general-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/phoenix-connectors/pull/125 |
   | Optional Tests | dupname asflicense javac javadoc unit xml compile |
   | uname | Linux 02027eaa43c3 5.15.0-94-generic #104-Ubuntu SMP Tue Jan 9 
15:25:40 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev/phoenix-connectors-personality.sh |
   | git revision | master / 48472b4 |
   | Default Java | Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08 |
   | mvninstall | 
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-125/4/artifact/yetus-general-check/output/branch-mvninstall-root.txt
 |
   | compile | 
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-125/4/artifact/yetus-general-check/output/branch-compile-root.txt
 |
   | javadoc | 
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-125/4/artifact/yetus-general-check/output/branch-javadoc-root.txt
 |
   | mvninstall | 
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-125/4/artifact/yetus-general-check/output/patch-mvninstall-root.txt
 |
   | compile | 
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-125/4/artifact/yetus-general-check/output/patch-compile-root.txt
 |
   | javac | 
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-125/4/artifact/yetus-general-check/output/patch-compile-root.txt
 |
   | javadoc | 
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-125/4/artifact/yetus-general-check/output/patch-javadoc-root.txt
 |
   | unit | 
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-125/4/artifact/yetus-general-check/output/patch-unit-root.txt
 |
   |  Test Results | 
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-125/4/testReport/
 |
   | Max. process+thread count | 52 (vs. ulimit of 30000) |
   | modules | C: . U: . |
   | Console output | 
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-125/4/console
 |
   | versions | git=2.7.4 maven=3.3.9 |
   | Powered by | Apache Yetus 0.12.0 https://yetus.apache.org |
   
   
   This message was automatically generated.
   
   




> Update Connectors to phoenix-thirdparty 2.1.0
> ---------------------------------------------
>
>                 Key: PHOENIX-7158
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-7158
>             Project: Phoenix
>          Issue Type: Improvement
>          Components: connectors
>            Reporter: Nihal Jain
>            Assignee: Nihal Jain
>            Priority: Major
>
> Phoenix-thirdparty has been released, see 
> [https://www.mail-archive.com/user@phoenix.apache.org/msg08204.html]
> {quote}The recent release has upgraded Guava to version 32.1.3-jre from the 
> previous 31.0.1-android version. Initially, the 4.x branch maintained 
> compatibility with Java 7, necessitating the use of the Android variant of 
> Guava. However, with the end-of-life (EOL) status of the 4.x branch, the move 
> to the standard JRE version of Guava signifies a shift in compatibility 
> standards
> {quote}
> It's time we bump up. Also, now PHOENIX-7116 is in place so we can pull this 
> to branch 5.1.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to