[kudu-CR] env util: Implement CreateDirsRecursively()

2017-01-05 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: env_util: Implement CreateDirsRecursively() .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/5618/2/src/kudu/util/env_util-test.cc File

[kudu-CR] env util: Implement CreateDirsRecursively()

2017-01-05 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: env_util: Implement CreateDirsRecursively() .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/5618/2/src/kudu/util/env_util.cc File

[kudu-CR] env util: Implement CreateDirsRecursively()

2017-01-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: env_util: Implement CreateDirsRecursively() .. Patch Set 2: (2 comments) general note: while I like fine-grained patches, for these simple utility functions I

[kudu-CR] env util: Implement CreateDirsRecursively()

2017-01-05 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5618 to look at the new patch set (#2). Change subject: env_util: Implement CreateDirsRecursively() .. env_util:

[kudu-CR] env util: Implement CreateDirsRecursively()

2017-01-05 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: env_util: Implement CreateDirsRecursively() .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5618/1/src/kudu/util/env_util.cc File

[kudu-CR] env util: Implement CreateDirsRecursively()

2017-01-05 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: env_util: Implement CreateDirsRecursively() .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/5618/1/src/kudu/util/env_util-test.cc File

[kudu-CR] env util: Implement CreateDirsRecursively()

2017-01-05 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: env_util: Implement CreateDirsRecursively() .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/5618/1/src/kudu/util/env_util.cc File

[kudu-CR] env util: Implement CreateDirsRecursively()

2017-01-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: env_util: Implement CreateDirsRecursively() .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/5618/1/src/kudu/util/env_util-test.cc File

[kudu-CR] env util: Implement CreateDirsRecursively()

2017-01-05 Thread Mike Percy (Code Review)
Hello Dinesh Bhat, Lars Volker, Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5618 to review the following change. Change subject: env_util: Implement CreateDirsRecursively()