[jira] [Commented] (HBASE-16035) Nested AutoCloseables might not all get closed

["Hadoop QA (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/HBASE-16035?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15332564#comment-15332564 ] 

Hadoop QA commented on HBASE-16035:
-----------------------------------

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:red}-1{color} | {color:red} pre-patch {color} | {color:red} 0m 0s {color} | {color:red} JAVA_HOME is not defined. {color} |
\\
\\
|| Subsystem || Report/Notes ||
| JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12810919/HBASE-16035-v1.patch |
| JIRA Issue | HBASE-16035 |
| Optional Tests |  asflicense  javac  javadoc  unit  findbugs  hadoopcheck  hbaseanti  checkstyle  compile  |
| uname | Linux pietas.apache.org 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | /home/jenkins/jenkins-slave/workspace/PreCommit-HBASE-Build/component/dev-support/hbase-personality.sh |
| git revision | master / bff35d6 |
| Console output | https://builds.apache.org/job/PreCommit-HBASE-Build/2235/console |
| Powered by | Apache Yetus 0.2.1   http://yetus.apache.org |


This message was automatically generated.



> Nested AutoCloseables might not all get closed
> ----------------------------------------------
>
>                 Key: HBASE-16035
>                 URL: https://issues.apache.org/jira/browse/HBASE-16035
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Sean Mackrory
>         Attachments: HBASE-16035-v1.patch
>
>
> Subtle problem in HBASE-15891:
> {code}try (A myA = new A(new B())){code}
> An exception thrown between B starting to open an A finishing initialization may not result in B being closed. A safer syntax would be:
> {code}try(B myB = new B(); A myA = newA(myB)){code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)