Skip to content

PHOENIX-6497 Remove embedded profile and always build phoenix-client-… - #1255

Closed
stoty wants to merge 1 commit into
apache:4.xfrom
stoty:PHOENIX-6497-4.x
Closed

PHOENIX-6497 Remove embedded profile and always build phoenix-client-…#1255
stoty wants to merge 1 commit into
apache:4.xfrom
stoty:PHOENIX-6497-4.x

Conversation

@stoty

Copy link
Copy Markdown
Contributor

…embedded

@stoty

Copy link
Copy Markdown
ContributorAuthor

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1255/1/console in case of problems.

@virajjasanivirajjasani left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, pending QA

@stoty

Copy link
Copy Markdown
ContributorAuthor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 38sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
-1 ❌test4tests0m 0sThe 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.
_ 4.x Compile Tests _
+1 💚mvninstall19m 53s4.x passed
+1 💚compile1m 56s4.x passed
+1 💚javadoc1m 18s4.x passed
_ Patch Compile Tests _
+1 💚mvninstall12m 0sthe patch passed
+1 💚compile2m 0sthe patch passed
+1 💚javac2m 0sthe patch passed
+1 💚shellcheck0m 2sThere were no new shellcheck issues.
+1 💚shelldocs0m 2sThere were no new shelldocs issues.
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 1sThe patch has no ill-formed XML file.
+1 💚javadoc1m 15sthe patch passed
_ Other Tests _
-1 ❌unit149m 35sroot in the patch failed.
+1 💚asflicense0m 13sThe patch does not generate ASF License warnings.
189m 36s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1255/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1255
Optional Testsdupname asflicense shellcheck shelldocs javac javadoc unit xml compile
unameLinux d54028f77b80 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revision4.x / 68ef271
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1255/1/artifact/yetus-general-check/output/patch-unit-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1255/1/testReport/
Max. process+thread count5874 (vs. ulimit of 30000)
modulesC: . U: .
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1255/1/console
versionsgit=2.7.4 maven=3.3.9 shellcheck=0.7.0
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@stotystoty closed this Jun 18, 2021
@stoty
stoty deleted the PHOENIX-6497-4.x branch July 29, 2021 14:43
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@stoty@virajjasani