[kudu-CR] Add some path / env related helper functions

2017-01-19 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: Add some path / env related helper functions .. Add some path / env related helper functions * path_util: Add SplitPath() helper function This function

[kudu-CR] Add some path / env related helper functions

2017-01-19 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add some path / env related helper functions .. Patch Set 4: Verified+1 Yeah, spurious failure. -- To view, visit http://gerrit.cloudera.org:8080/5618 To unsubscr

[kudu-CR] Add some path / env related helper functions

2017-01-19 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add some path / env related helper functions .. Patch Set 4: Code-Review+2 Test failure looks spurious. -- To view, visit http://gerrit.cloudera.org:8080/5618 To

[kudu-CR] Add some path / env related helper functions

2017-01-19 Thread Mike Percy (Code Review)
Hello Adar Dembo, 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 (#4). Change subject: Add some path / env related helper functions .. Add

[kudu-CR] Add some path / env related helper functions

2017-01-19 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add some path / env related helper functions .. Patch Set 3: (7 comments) http://gerrit.cloudera.org:8080/#/c/5618/3/src/kudu/util/env_util-test.cc File src/kudu/u

[kudu-CR] Add some path / env related helper functions

2017-01-09 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: Add some path / env related helper functions .. Patch Set 3: -Code-Review (1 comment) http://gerrit.cloudera.org:8080/#/c/5618/3/src/kudu/util/env_posix.cc File s

[kudu-CR] Add some path / env related helper functions

2017-01-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add some path / env related helper functions .. Patch Set 3: (4 comments) http://gerrit.cloudera.org:8080/#/c/5618/3/src/kudu/util/env_posix.cc File src/kudu/util/

[kudu-CR] Add some path / env related helper functions

2017-01-06 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: Add some path / env related helper functions .. Patch Set 3: Code-Review+1 (5 comments) http://gerrit.cloudera.org:8080/#/c/5618/3/src/kudu/util/env_util-test.cc

[kudu-CR] Add some path / env related helper functions

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 (#3). Change subject: Add some path / env related helper functions .. Add some path /