Skip to content

PHOENIX-6183 Page size tests are not propagating test override valuees to server - #914

Merged
swaroopak merged 1 commit into
apache:masterfrom
tkhurana:PHOENIX-6183
Oct 12, 2020
Merged

PHOENIX-6183 Page size tests are not propagating test override valuees to server#914
swaroopak merged 1 commit into
apache:masterfrom
tkhurana:PHOENIX-6183

Conversation

@tkhurana

Copy link
Copy Markdown
Contributor

Tests are creating a new conf object and setting the
INDEX_REBUILD_PAGE_SIZE_IN_ROWS parameter. Pass that conf object to the
IndexTool.

… to server
Tests are creating a new conf object and setting the
INDEX_REBUILD_PAGE_SIZE_IN_ROWS parameter. Pass that conf object to the
IndexTool.
@tkhurana

Copy link
Copy Markdown
ContributorAuthor

@stoty

stoty commented Oct 8, 2020

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 34sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 1sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+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.
_ master Compile Tests _
+1 💚mvninstall11m 1smaster passed
+1 💚compile0m 57smaster passed
+1 💚checkstyle0m 29smaster passed
+1 💚javadoc0m 44smaster passed
+0 🆗spotbugs3m 1sphoenix-core in master has 973 extant spotbugs warnings.
_ Patch Compile Tests _
+1 💚mvninstall7m 1sthe patch passed
+1 💚compile0m 53sthe patch passed
+1 💚javac0m 53sthe patch passed
+1 💚checkstyle0m 32sthe patch passed
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 47sthe patch passed
+1 💚spotbugs3m 8sthe patch passed
_ Other Tests _
-1 ❌unit102m 21sphoenix-core in the patch failed.
+1 💚asflicense0m 26sThe patch does not generate ASF License warnings.
134m 28s
SubsystemReport/Notes
DockerClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-914/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#914
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile
unameLinux 8a979fb81ad7 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revisionmaster / 92e5908
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-914/1/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-914/1/testReport/
Max. process+thread count6808 (vs. ulimit of 30000)
modulesC: phoenix-core U: phoenix-core
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-914/1/console
versionsgit=2.7.4 maven=3.3.9 spotbugs=4.1.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@swaroopak
swaroopak merged commit 9d93f16 into apache:masterOct 12, 2020
swaroopak pushed a commit that referenced this pull request Oct 12, 2020
… to server (#914)
Tests are creating a new conf object and setting the
INDEX_REBUILD_PAGE_SIZE_IN_ROWS parameter. Pass that conf object to the
IndexTool.
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.

3 participants

@tkhurana@stoty@swaroopak