Skip to content

HBASE-22611 Close the DataOutputStream object - #325

Merged
saintstack merged 1 commit into
apache:masterfrom
yuliangwan:dev
Jun 24, 2019
Merged

HBASE-22611 Close the DataOutputStream object#325
saintstack merged 1 commit into
apache:masterfrom
yuliangwan:dev

Conversation

@yuliangwan

Copy link
Copy Markdown
Contributor

DataOutputStream is not closed after used.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec98Docker mode activated.
_ Prechecks _
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The 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 _
+1mvninstall391master passed
+1compile38master passed
+1checkstyle42master passed
+1shadedjars409branch has no errors when building our shaded downstream artifacts.
+1findbugs73master passed
+1javadoc34master passed
_ Patch Compile Tests _
+1mvninstall384the patch passed
+1compile40the patch passed
+1javac40the patch passed
+1checkstyle47the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars409patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1154Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1findbugs68the patch passed
+1javadoc29the patch passed
_ Other Tests _
+1unit225hbase-common in the patch passed.
+1asflicense13The patch does not generate ASF License warnings.
3914
SubsystemReport/Notes
DockerClient=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-325/1/artifact/out/Dockerfile
GITHUB PR#325
Optional Testsdupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 861758f6c0cb 4.4.0-131-generic #157~14.04.1-Ubuntu SMP Fri Jul 13 08:53:17 UTC 2018 x86_64 GNU/Linux
Build toolmaven
Personality/testptch/patchprocess/precommit/personality/provided.sh
git revisionmaster / c7c6140
mavenversion: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
Default Java1.8.0_181
findbugsv3.1.11
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-325/1/testReport/
Max. process+thread count254 (vs. ulimit of 10000)
modulesC: hbase-common U: hbase-common
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-325/1/console
Powered byApache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@saintstack
saintstack merged commit 841b319 into apache:masterJun 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.

3 participants

@yuliangwan@Apache-HBase@saintstack