[GitHub] storm pull request #2005: STORM-2414 Skip checking ACL when clearing already...

2017-03-16 Thread arunmahadevan
Github user arunmahadevan commented on a diff in the pull request: https://github.com/apache/storm/pull/2005#discussion_r106585741 --- Diff: storm-core/src/jvm/org/apache/storm/blobstore/LocalFsBlobStore.java --- @@ -219,17 +219,44 @@ public void setBlobMeta(String key, SettableBlo

[GitHub] storm pull request #2005: STORM-2414 Skip checking ACL when clearing already...

2017-03-16 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/2005#discussion_r106584890 --- Diff: storm-core/src/jvm/org/apache/storm/blobstore/LocalFsBlobStore.java --- @@ -219,9 +219,20 @@ public void setBlobMeta(String key, SettableBlobMe

[GitHub] storm pull request #2005: STORM-2414 Skip checking ACL when clearing already...

2017-03-16 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/2005#discussion_r106584346 --- Diff: storm-core/src/jvm/org/apache/storm/blobstore/LocalFsBlobStore.java --- @@ -219,17 +219,44 @@ public void setBlobMeta(String key, SettableBlobM

[GitHub] storm pull request #2005: STORM-2414 Skip checking ACL when clearing already...

2017-03-16 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/2005#discussion_r106584243 --- Diff: storm-core/src/jvm/org/apache/storm/blobstore/LocalFsBlobStore.java --- @@ -219,17 +219,44 @@ public void setBlobMeta(String key, SettableBlobM

[GitHub] storm pull request #2005: STORM-2414 Skip checking ACL when clearing already...

2017-03-16 Thread arunmahadevan
Github user arunmahadevan commented on a diff in the pull request: https://github.com/apache/storm/pull/2005#discussion_r106455903 --- Diff: storm-core/src/jvm/org/apache/storm/blobstore/LocalFsBlobStore.java --- @@ -219,17 +219,44 @@ public void setBlobMeta(String key, SettableBlo

[GitHub] storm pull request #2005: STORM-2414 Skip checking ACL when clearing already...

2017-03-16 Thread satishd
Github user satishd commented on a diff in the pull request: https://github.com/apache/storm/pull/2005#discussion_r106440281 --- Diff: storm-core/src/jvm/org/apache/storm/blobstore/LocalFsBlobStore.java --- @@ -219,17 +219,44 @@ public void setBlobMeta(String key, SettableBlobMeta

[GitHub] storm pull request #2005: STORM-2414 Skip checking ACL when clearing already...

2017-03-16 Thread satishd
Github user satishd commented on a diff in the pull request: https://github.com/apache/storm/pull/2005#discussion_r106439220 --- Diff: storm-core/src/jvm/org/apache/storm/blobstore/LocalFsBlobStore.java --- @@ -219,9 +219,20 @@ public void setBlobMeta(String key, SettableBlobMeta

[GitHub] storm pull request #2005: STORM-2414 Skip checking ACL when clearing already...

2017-03-15 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/2005#discussion_r106349077 --- Diff: storm-core/src/jvm/org/apache/storm/blobstore/LocalFsBlobStore.java --- @@ -219,9 +219,20 @@ public void setBlobMeta(String key, SettableBlobMe

[GitHub] storm pull request #2005: STORM-2414 Skip checking ACL when clearing already...

2017-03-15 Thread arunmahadevan
Github user arunmahadevan commented on a diff in the pull request: https://github.com/apache/storm/pull/2005#discussion_r106346689 --- Diff: storm-core/src/jvm/org/apache/storm/blobstore/LocalFsBlobStore.java --- @@ -219,9 +219,20 @@ public void setBlobMeta(String key, SettableBlob

[GitHub] storm pull request #2005: STORM-2414 Skip checking ACL when clearing already...

2017-03-15 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/2005#discussion_r106346290 --- Diff: storm-core/src/jvm/org/apache/storm/blobstore/LocalFsBlobStore.java --- @@ -219,9 +219,20 @@ public void setBlobMeta(String key, SettableBlobMe

[GitHub] storm pull request #2005: STORM-2414 Skip checking ACL when clearing already...

2017-03-15 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/2005#discussion_r106346237 --- Diff: storm-core/src/jvm/org/apache/storm/blobstore/LocalFsBlobStore.java --- @@ -219,9 +219,20 @@ public void setBlobMeta(String key, SettableBlobMe

[GitHub] storm pull request #2005: STORM-2414 Skip checking ACL when clearing already...

2017-03-15 Thread satishd
Github user satishd commented on a diff in the pull request: https://github.com/apache/storm/pull/2005#discussion_r106343405 --- Diff: storm-core/src/jvm/org/apache/storm/blobstore/BlobStore.java --- @@ -128,6 +128,15 @@ public abstract void deleteBlob(String key, Subject w

[GitHub] storm pull request #2005: STORM-2414 Skip checking ACL when clearing already...

2017-03-15 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/2005#discussion_r106342906 --- Diff: storm-core/src/jvm/org/apache/storm/blobstore/LocalFsBlobStore.java --- @@ -231,6 +231,18 @@ public void deleteBlob(String key, Subject who) th

[GitHub] storm pull request #2005: STORM-2414 Skip checking ACL when clearing already...

2017-03-15 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/2005#discussion_r106342738 --- Diff: storm-core/src/jvm/org/apache/storm/blobstore/BlobStore.java --- @@ -128,6 +128,15 @@ public abstract void deleteBlob(String key, Subje

[GitHub] storm pull request #2005: STORM-2414 Skip checking ACL when clearing already...

2017-03-15 Thread arunmahadevan
Github user arunmahadevan commented on a diff in the pull request: https://github.com/apache/storm/pull/2005#discussion_r106340804 --- Diff: storm-core/src/jvm/org/apache/storm/blobstore/LocalFsBlobStore.java --- @@ -231,6 +231,18 @@ public void deleteBlob(String key, Subject who)

[GitHub] storm pull request #2005: STORM-2414 Skip checking ACL when clearing already...

2017-03-15 Thread satishd
Github user satishd commented on a diff in the pull request: https://github.com/apache/storm/pull/2005#discussion_r106334482 --- Diff: storm-core/src/jvm/org/apache/storm/blobstore/BlobStore.java --- @@ -128,6 +128,15 @@ public abstract void deleteBlob(String key, Subject w

[GitHub] storm pull request #2005: STORM-2414 Skip checking ACL when clearing already...

2017-03-14 Thread HeartSaVioR
GitHub user HeartSaVioR opened a pull request: https://github.com/apache/storm/pull/2005 STORM-2414 Skip checking ACL when clearing already removed blob from … …local / HDFS blobstore Syncing up blobs throws KeyNotFoundException when deleting blobs which are already del