Skip to content

PHOENIX-6320 support hbase profile param at the release script - #1093

Closed
yanxinyi wants to merge 1 commit into
apache:4.16from
yanxinyi:PHOENIX-6320-4.16
Closed

PHOENIX-6320 support hbase profile param at the release script#1093
yanxinyi wants to merge 1 commit into
apache:4.16from
yanxinyi:PHOENIX-6320-4.16

Conversation

@yanxinyi

Copy link
Copy Markdown
Contributor

No description provided.

@yanxinyi

Copy link
Copy Markdown
ContributorAuthor

@stoty@ChinmaySKulkarni , can you guys take a look, thanks!

@stoty

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 19sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 1sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ 4.16 Compile Tests _
+0 🆗mvndep4m 52sMaven dependency ordering for branch
_ Patch Compile Tests _
+0 🆗mvndep0m 7sMaven dependency ordering for patch
-1 ❌shellcheck0m 1sThe patch generated 2 new + 52 unchanged - 0 fixed = 54 total (was 52)
+1 💚shelldocs0m 2sThere were no new shelldocs issues.
+1 💚whitespace0m 0sThe patch has no whitespace issues.
_ Other Tests _
+0 🆗asflicense0m 0sASF License check generated no output?
6m 47s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1093/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1093
Optional Testsdupname asflicense shellcheck shelldocs
unameLinux 0f736a22b4d0 4.15.0-126-generic #129-Ubuntu SMP Mon Nov 23 18:53:38 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revision4.16 / dc1118a
shellcheckhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1093/1/artifact/yetus-general-check/output/diff-patch-shellcheck.txt
Max. process+thread count47 (vs. ulimit of 30000)
modulesC: U:
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1093/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.

@stoty

Copy link
Copy Markdown
Contributor

The change looks OK, but please consider using the script in /dev/create-release instead.
(more details in the 4.16 thread on the dev list)

@stoty

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec5m 29sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ 4.16 Compile Tests _
+0 🆗mvndep4m 54sMaven dependency ordering for branch
_ Patch Compile Tests _
+0 🆗mvndep0m 8sMaven dependency ordering for patch
-1 ❌shellcheck0m 2sThe patch generated 2 new + 52 unchanged - 0 fixed = 54 total (was 52)
+1 💚shelldocs0m 0sThere were no new shelldocs issues.
+1 💚whitespace0m 0sThe patch has no whitespace issues.
_ Other Tests _
+0 🆗asflicense0m 0sASF License check generated no output?
10m 55s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1093/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1093
Optional Testsdupname asflicense shellcheck shelldocs
unameLinux 3d45a778d0ea 4.15.0-126-generic #129-Ubuntu SMP Mon Nov 23 18:53:38 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revision4.16 / 90b99b0
shellcheckhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1093/1/artifact/yetus-general-check/output/diff-patch-shellcheck.txt
Max. process+thread count47 (vs. ulimit of 30000)
modulesC: U:
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1093/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.

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

@yanxinyi@stoty