phoenix-connectors icon indicating copy to clipboard operation
phoenix-connectors copied to clipboard

PHOENIX-7158 Update Connectors to phoenix-thirdparty 2.1.0

Open NihalJain opened this issue 2 years ago • 9 comments

NihalJain avatar Dec 22 '23 11:12 NihalJain

:broken_heart: -1 overall

Vote Subsystem Runtime Comment
+0 :ok: reexec 6m 12s 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 6s 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 1s The patch has no ill-formed XML file.
-1 :x: javadoc 0m 6s root in the patch failed.
_ Other Tests _
-1 :x: unit 0m 6s root in the patch failed.
+1 :green_heart: asflicense 0m 19s The patch does not generate ASF License warnings.
7m 38s
Subsystem Report/Notes
Docker ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-125/1/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 e60d81ed8364 5.15.0-88-generic #98-Ubuntu SMP Mon Oct 2 15:18:56 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev/phoenix-connectors-personality.sh
git revision master / 80abdb4
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/1/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/1/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/1/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/1/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/1/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/1/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/1/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/1/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/1/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/1/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.

stoty avatar Dec 22 '23 11:12 stoty

Hi @stoty could you please help with this one? I am not sure why jenkins is failing here. Also how do we build connectors: plainmvn clean install? For me I cannot build in local as the project uses log4j:log4j which is vulnerable and banned in my org. I am working on a patch to completely remove that dependency, will raise a patch soon.

NihalJain avatar Jan 06 '24 16:01 NihalJain

Raised https://issues.apache.org/jira/browse/PHOENIX-7169

NihalJain avatar Jan 06 '24 17:01 NihalJain

@NihalJain The broken Jenkins CI is known issue, see https://issues.apache.org/jira/browse/PHOENIX-7129 . The current workaround is ignoring the Apache CI results, and relying on the github actions check result.

I have replied to the log4j issue on its ticket.

stoty avatar Jan 08 '24 06:01 stoty

@NihalJain The broken Jenkins CI is known issue, see https://issues.apache.org/jira/browse/PHOENIX-7129 . The current workaround is ignoring the Apache CI results, and relying on the github actions check result.

Thank you for your response @stoty. Will look at github actions results, going ahead.

I have replied to the log4j issue on its ticket.

Also, let me try to see if I can fix log4j issue.

NihalJain avatar Jan 09 '24 07:01 NihalJain

:broken_heart: -1 overall

Vote Subsystem Runtime Comment
+0 :ok: reexec 6m 23s 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 6s root in master failed.
_ Patch Compile Tests _
-1 :x: mvninstall 0m 6s root in the patch failed.
-1 :x: compile 0m 5s root in the patch failed.
-1 :x: javac 0m 5s root in the patch failed.
+1 :green_heart: whitespace 0m 0s The patch has no whitespace issues.
+1 :green_heart: xml 0m 1s The patch has no ill-formed XML file.
-1 :x: javadoc 0m 6s root in the patch failed.
_ Other Tests _
-1 :x: unit 0m 6s root in the patch failed.
+1 :green_heart: asflicense 0m 19s The patch does not generate ASF License warnings.
7m 47s
Subsystem Report/Notes
Docker ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-125/2/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 3d893a7ffbba 5.15.0-88-generic #98-Ubuntu SMP Mon Oct 2 15:18:56 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev/phoenix-connectors-personality.sh
git revision master / 60d4172
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/2/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/2/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/2/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/2/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/2/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/2/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/2/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/2/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/2/testReport/
Max. process+thread count 47 (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/2/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.

stoty avatar Jan 09 '24 18:01 stoty

Hi please let me know if anything more is needed here. Also I have a WIP patch for log4j fix. Will raise soon.

CC: @stoty

NihalJain avatar Feb 09 '24 07:02 NihalJain

:broken_heart: -1 overall

Vote Subsystem Runtime Comment
+0 :ok: reexec 6m 40s 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 6s 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 1s The patch has no ill-formed XML file.
-1 :x: javadoc 0m 6s root in the patch failed.
_ Other Tests _
-1 :x: unit 0m 5s root in the patch failed.
+1 :green_heart: asflicense 0m 19s The patch does not generate ASF License warnings.
8m 4s
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/3/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 757e406b9ff0 5.15.0-88-generic #98-Ubuntu SMP Mon Oct 2 15:18:56 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev/phoenix-connectors-personality.sh
git revision master / 6d57726
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/3/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/3/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/3/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/3/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/3/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/3/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/3/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/3/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/3/testReport/
Max. process+thread count 47 (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/3/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.

stoty avatar Feb 09 '24 09:02 stoty

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

stoty avatar Mar 05 '24 07:03 stoty

Done as part of the Phoenix 5.2 support patch

stoty avatar Jul 25 '24 06:07 stoty