[GitHub] storm issue #2336: STORM-2750：fix double_checked locking

Github user xiaoyong03 commented on the issue: https://github.com/apache/storm/pull/2336 thanks @srdo . I squashd my commits to one. ---

[GitHub] storm issue #2336: STORM-2750：fix double_checked locking

Github user xiaoyong03 commented on the issue: https://github.com/apache/storm/pull/2336 @revans2 this is found in our other projec, and I find there is the same bug in strom while reading source code with static analysis. ---

[GitHub] storm issue #2336: STORM-2750：fix double_checked locking

Github user xiaoyong03 commented on the issue: https://github.com/apache/storm/pull/2336 @srdo I have created an issue for this at https://issues.apache.org/jira and then rename the PR and commit message to contain the issue number as your request. thanks for the review ---

[GitHub] storm pull request #2336: STORM-2750：fix HBaseSecurityUtil double_checked ...

GitHub user xiaoyong03 opened a pull request: https://github.com/apache/storm/pull/2336 STORM-2750：fix HBaseSecurityUtil double_checked locking update HBaseSecurityUtil singleton to fix double_checked locking See url link for details: http://www.cs.umd.edu/~pugh/java

[GitHub] storm pull request #2335: STORM-2750：fix HBaseSecurityUtil double_checked ...

Github user xiaoyong03 closed the pull request at: https://github.com/apache/storm/pull/2335 ---

[GitHub] storm issue #2335: STORM-2750：fix HBaseSecurityUtil double_checked locking

Github user xiaoyong03 commented on the issue: https://github.com/apache/storm/pull/2335 @srdo I have created an issue for this at https://issues.apache.org/jira and then rename the PR and commit message to contain the issue number as your request. thanks for the review ---

[GitHub] storm pull request #2335: STORM-2750：fix HBaseSecurityUtil double_checked ...

GitHub user xiaoyong03 opened a pull request: https://github.com/apache/storm/pull/2335 STORM-2750：fix HBaseSecurityUtil double_checked locking update HBaseSecurityUtil singleton to fix double_checked locking. See url link for details: http://www.cs.umd.edu/~pugh/java

[GitHub] storm pull request #2333: STORM-2750：fix HBaseSecurityUtil double_checked ...

Github user xiaoyong03 closed the pull request at: https://github.com/apache/storm/pull/2333 ---

[GitHub] storm pull request #2333: fix HBaseSecurityUtil double_checked,

Github user xiaoyong03 closed the pull request at: https://github.com/apache/storm/pull/2333 ---

[GitHub] storm pull request #2333: fix HBaseSecurityUtil double_checked,

GitHub user xiaoyong03 reopened a pull request: https://github.com/apache/storm/pull/2333 fix HBaseSecurityUtildouble_checked, update HBaseSecurityUtil singleton to fix double_checked, http://www.cs.umd.edu/~pugh/java/memoryModel/DoubleCheckedLocking.html You can merge this

[GitHub] storm issue #2333: fix HBaseSecurityUtil double_checked,

Github user xiaoyong03 commented on the issue: https://github.com/apache/storm/pull/2333 thanks for review ---

[GitHub] storm pull request #2333: fix HBaseSecurityUtil double_checked,

GitHub user xiaoyong03 opened a pull request: https://github.com/apache/storm/pull/2333 fix HBaseSecurityUtildouble_checked, update HBaseSecurityUtil singleton to fix double_checked, http://www.cs.umd.edu/~pugh/java/memoryModel/DoubleCheckedLocking.html You can merge this