Skip to content

HBASE-22299 Documentation has incorrect default number of versions - #187

Merged
petersomogyi merged 1 commit into
apache:masterfrom
anis016:HBASE-22299
Apr 24, 2019
Merged

HBASE-22299 Documentation has incorrect default number of versions#187
petersomogyi merged 1 commit into
apache:masterfrom
anis016:HBASE-22299

Conversation

@anis016

@anis016anis016 commented Apr 24, 2019

Copy link
Copy Markdown
Contributor

Corrected the default value as 1. Also updated the deprecated HColumnDescriptor class and added ColumnFamilyDescriptorBuilder in the documentation.

Fixes: HBASE-22299

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec68Docker mode activated.
_ Prechecks _
+1@author0The patch does not contain any @author tags.
_ master Compile Tests _
+1mvninstall376master passed
0refguide529branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
_ Patch Compile Tests _
+1mvninstall272the patch passed
+1whitespace0The patch has no whitespace issues.
0refguide469patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
_ Other Tests _
+1asflicense17The patch does not generate ASF License warnings.
1798
SubsystemReport/Notes
DockerClient=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-187/1/artifact/out/Dockerfile
GITHUB PR#187
Optional Testsdupname asflicense refguide
unameLinux 4122f04ddf16 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux
Build toolmaven
Personality/testptch/patchprocess/precommit/personality/provided.sh
git revisionmaster / d3bf9c0
mavenversion: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
refguidehttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-187/1/artifact/out/branch-site/book.html
refguidehttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-187/1/artifact/out/patch-site/book.html
Max. process+thread count96 (vs. ulimit of 10000)
modulesC: . U: .
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-187/1/console
Powered byApache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@petersomogyipetersomogyi 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.

Thanks for the patch @anis016!

@petersomogyi
petersomogyi merged commit e39f7dc into apache:masterApr 24, 2019
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.

4 participants

@anis016@Apache-HBase@HorizonNet@petersomogyi