[GitHub] drill pull request #652: DRILL-4990:Use new HDFS API access instead of listS...

2017-11-06 Thread ppadma
Github user ppadma commented on a diff in the pull request: https://github.com/apache/drill/pull/652#discussion_r149173203 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/WorkspaceSchemaFactory.java --- @@ -151,17 +152,32 @@ public WorkspaceSchemaFactory(

[GitHub] drill pull request #652: DRILL-4990:Use new HDFS API access instead of listS...

2017-11-05 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/652#discussion_r148991210 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/WorkspaceSchemaFactory.java --- @@ -151,17 +152,32 @@ public WorkspaceSchemaFactory(

[GitHub] drill pull request #652: DRILL-4990:Use new HDFS API access instead of listS...

2017-11-05 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/652#discussion_r148991415 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/WorkspaceSchemaFactory.java --- @@ -151,17 +152,32 @@ public WorkspaceSchemaFactory(

[GitHub] drill pull request #652: DRILL-4990:Use new HDFS API access instead of listS...

2016-11-15 Thread ppadma
Github user ppadma commented on a diff in the pull request: https://github.com/apache/drill/pull/652#discussion_r88097089 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/WorkspaceSchemaFactory.java --- @@ -151,15 +152,11 @@ public WorkspaceSchemaFactory(

[GitHub] drill pull request #652: DRILL-4990:Use new HDFS API access instead of listS...

2016-11-15 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/652#discussion_r88072130 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/WorkspaceSchemaFactory.java --- @@ -151,15 +152,11 @@ public WorkspaceSchemaFactory(

[GitHub] drill pull request #652: DRILL-4990:Use new HDFS API access instead of listS...

2016-11-15 Thread sohami
Github user sohami commented on a diff in the pull request: https://github.com/apache/drill/pull/652#discussion_r88072322 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/WorkspaceSchemaFactory.java --- @@ -151,15 +152,11 @@ public WorkspaceSchemaFactory(

[GitHub] drill pull request #652: DRILL-4990:Use new HDFS API access instead of listS...

2016-11-14 Thread ppadma
GitHub user ppadma opened a pull request: https://github.com/apache/drill/pull/652 DRILL-4990:Use new HDFS API access instead of listStatus to check if … …users have permissions to access workspace. Manually tested the fix with impersonation enabled. All unit and