You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Taras Bobrovytsky (Code Review)" <ge...@cloudera.org> on 2016/06/23 20:03:41 UTC

[Impala-CR](cdh5-2.6.0 5.8.0) IMPALA-3698: Fix Isilon permissions test

Taras Bobrovytsky has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/3471

Change subject: IMPALA-3698: Fix Isilon permissions test
......................................................................

IMPALA-3698: Fix Isilon permissions test

Isilon OneFS 8.0 introduces a regression where a
ConnectionRefusedException is thrown if the table directory permission
is set to 444. This patch fixes the problem by changing the table
directory permissions to 544 for Isilon in the read only table test.

Change-Id: Ie5ed082133ad3ca35a7f68fa4be118a0e59270b8
Reviewed-on: http://gerrit.cloudera.org:8080/3433
Reviewed-by: David Knupp <dk...@cloudera.com>
Tested-by: Taras Bobrovytsky <tb...@cloudera.com>
(cherry picked from commit b45d56cd78d78407d8b036ef6fce5cf82e8ab7a4)
---
M tests/metadata/test_hdfs_permissions.py
1 file changed, 6 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/71/3471/1
-- 
To view, visit http://gerrit.cloudera.org:8080/3471
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie5ed082133ad3ca35a7f68fa4be118a0e59270b8
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-2.6.0_5.8.0
Gerrit-Owner: Taras Bobrovytsky <tb...@cloudera.com>

[Impala-CR](cdh5-2.6.0 5.8.0) IMPALA-3698: Fix Isilon permissions test

Posted by "Taras Bobrovytsky (Code Review)" <ge...@cloudera.org>.
Taras Bobrovytsky has submitted this change and it was merged.

Change subject: IMPALA-3698: Fix Isilon permissions test
......................................................................


IMPALA-3698: Fix Isilon permissions test

Isilon OneFS 8.0 introduces a regression where a
ConnectionRefusedException is thrown if the table directory permission
is set to 444. This patch fixes the problem by changing the table
directory permissions to 544 for Isilon in the read only table test.

Change-Id: Ie5ed082133ad3ca35a7f68fa4be118a0e59270b8
Reviewed-on: http://gerrit.cloudera.org:8080/3433
Reviewed-by: David Knupp <dk...@cloudera.com>
Tested-by: Taras Bobrovytsky <tb...@cloudera.com>
(cherry picked from commit b45d56cd78d78407d8b036ef6fce5cf82e8ab7a4)
Reviewed-on: http://gerrit.cloudera.org:8080/3471
Reviewed-by: Taras Bobrovytsky <tb...@cloudera.com>
---
M tests/metadata/test_hdfs_permissions.py
1 file changed, 6 insertions(+), 2 deletions(-)

Approvals:
  Taras Bobrovytsky: Looks good to me, approved; Verified



-- 
To view, visit http://gerrit.cloudera.org:8080/3471
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie5ed082133ad3ca35a7f68fa4be118a0e59270b8
Gerrit-PatchSet: 2
Gerrit-Project: Impala
Gerrit-Branch: cdh5-2.6.0_5.8.0
Gerrit-Owner: Taras Bobrovytsky <tb...@cloudera.com>
Gerrit-Reviewer: Taras Bobrovytsky <tb...@cloudera.com>

[Impala-CR](cdh5-2.6.0 5.8.0) IMPALA-3698: Fix Isilon permissions test

Posted by "Taras Bobrovytsky (Code Review)" <ge...@cloudera.org>.
Taras Bobrovytsky has posted comments on this change.

Change subject: IMPALA-3698: Fix Isilon permissions test
......................................................................


Patch Set 1: Code-Review+2 Verified+1

Cherry picked to 5.8.0 branch, forwarding the +2 and verified.

-- 
To view, visit http://gerrit.cloudera.org:8080/3471
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie5ed082133ad3ca35a7f68fa4be118a0e59270b8
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-2.6.0_5.8.0
Gerrit-Owner: Taras Bobrovytsky <tb...@cloudera.com>
Gerrit-Reviewer: Taras Bobrovytsky <tb...@cloudera.com>
Gerrit-HasComments: No