[Impala-ASF-CR] IMPALA-7071: make get fs path() idempotent

2018-06-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10517 ) Change subject: IMPALA-7071: make get_fs_path() idempotent .. IMPALA-7071: make get_fs_path() idempotent This avoids

[Impala-ASF-CR] IMPALA-7071: make get fs path() idempotent

2018-06-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10517 ) Change subject: IMPALA-7071: make get_fs_path() idempotent .. Patch Set 4: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7071: make get fs path() idempotent

2018-06-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10517 ) Change subject: IMPALA-7071: make get_fs_path() idempotent .. Patch Set 4: Build started:

[Impala-ASF-CR] IMPALA-7071: make get fs path() idempotent

2018-06-07 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10517 ) Change subject: IMPALA-7071: make get_fs_path() idempotent .. Patch Set 3: Sure -- To view, visit http://gerrit.cloudera.org:8080/10517

[Impala-ASF-CR] IMPALA-7071: make get fs path() idempotent

2018-06-07 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10517 ) Change subject: IMPALA-7071: make get_fs_path() idempotent .. Patch Set 4: Code-Review+2 Carry +2 -- To view, visit

[Impala-ASF-CR] IMPALA-7071: make get fs path() idempotent

2018-06-07 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10517 ) Change subject: IMPALA-7071: make get_fs_path() idempotent .. Patch Set 3: Do you want to go ahead and merge this? Seems low risk.

[Impala-ASF-CR] IMPALA-7071: make get fs path() idempotent

2018-05-25 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/10517 ) Change subject: IMPALA-7071: make get_fs_path() idempotent .. Patch Set 3: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7071: make get fs path() idempotent

2018-05-25 Thread Dan Hecht (Code Review)
Hello Michael Brown, Sailesh Mukil, David Knupp, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10517 to look at the new patch set (#3). Change subject: IMPALA-7071: make get_fs_path() idempotent

[Impala-ASF-CR] IMPALA-7071: make get fs path() idempotent

2018-05-25 Thread David Knupp (Code Review)
David Knupp has posted comments on this change. ( http://gerrit.cloudera.org:8080/10517 ) Change subject: IMPALA-7071: make get_fs_path() idempotent .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-7071: make get fs path() idempotent

2018-05-25 Thread Dan Hecht (Code Review)
Hello Michael Brown, Sailesh Mukil, David Knupp, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10517 to look at the new patch set (#2). Change subject: IMPALA-7071: make get_fs_path() idempotent

[Impala-ASF-CR] IMPALA-7071: make get fs path() idempotent

2018-05-25 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10517 ) Change subject: IMPALA-7071: make get_fs_path() idempotent .. Patch Set 1: (2 comments)

[Impala-ASF-CR] IMPALA-7071: make get fs path() idempotent

2018-05-25 Thread David Knupp (Code Review)
David Knupp has posted comments on this change. ( http://gerrit.cloudera.org:8080/10517 ) Change subject: IMPALA-7071: make get_fs_path() idempotent .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-7071: make get fs path() idempotent

2018-05-25 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/10517 ) Change subject: IMPALA-7071: make get_fs_path() idempotent .. Patch Set 1: (1 comment) Looks fine.

[Impala-ASF-CR] IMPALA-7071: make get fs path() idempotent

2018-05-25 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/10517 ) Change subject: IMPALA-7071: make get_fs_path() idempotent .. Patch Set 1: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7071: make get fs path() idempotent

2018-05-25 Thread Dan Hecht (Code Review)
Dan Hecht has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10517 Change subject: IMPALA-7071: make get_fs_path() idempotent .. IMPALA-7071: make get_fs_path() idempotent This avoids future errors like