[kudu-CR] env: add ReadV() API

2017-05-03 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: env: add ReadV() API .. env: add ReadV() API Adds ReadV() methods to RWFile and RandomAccessFile that allows reading data into multiple Slices in one call.

[kudu-CR] env: add ReadV() API

2017-05-03 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: env: add ReadV() API .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6779 To unsubscribe, visit http://gerrit.cloudera.org:8080/set

[kudu-CR] env: add ReadV() API

2017-05-03 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: env: add ReadV() API .. Patch Set 6: (5 comments) http://gerrit.cloudera.org:8080/#/c/6779/6/src/kudu/util/env-test.cc File src/kudu/util/env-test.cc: Line 476:

[kudu-CR] env: add ReadV() API

2017-05-03 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6779 to look at the new patch set (#7). Change subject: env: add ReadV() API .. env: add ReadV() API Adds ReadV() metho

[kudu-CR] env: add ReadV() API

2017-05-03 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: env: add ReadV() API .. Patch Set 6: (5 comments) http://gerrit.cloudera.org:8080/#/c/6779/6/src/kudu/util/env-test.cc File src/kudu/util/env-test.cc: PS6, Line 4

[kudu-CR] env: add ReadV() API

2017-05-03 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6779 to look at the new patch set (#6). Change subject: env: add ReadV() API .. env: add ReadV() API Adds ReadV() metho

[kudu-CR] env: add ReadV() API

2017-05-03 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: env: add ReadV() API .. Patch Set 3: I need to add handling in the case the passed in vector exceeds IOV_MAX yet. -- To view, visit http://gerrit.cloudera.org:80

[kudu-CR] env: add ReadV() API

2017-05-03 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6779 to look at the new patch set (#5). Change subject: env: add ReadV() API .. env: add ReadV() API Adds ReadV() metho

[kudu-CR] env: add ReadV() API

2017-05-03 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6779 to look at the new patch set (#4). Change subject: env: add ReadV() API .. env: add ReadV() API Adds ReadV() metho

[kudu-CR] env: add ReadV() API

2017-05-03 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: env: add ReadV() API .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/6779/2/src/kudu/util/env_posix.cc File src/kudu/util/env_posix.cc: Line 330

[kudu-CR] env: add ReadV() API

2017-05-03 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6779 to look at the new patch set (#3). Change subject: env: add ReadV() API .. env: add ReadV() API Adds ReadV() metho

[kudu-CR] env: add ReadV() API

2017-05-02 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: env: add ReadV() API .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/6779/2/src/kudu/util/env_posix.cc File src/kudu/util/env_posix.cc: Line 330:

[kudu-CR] env: add ReadV() API

2017-05-02 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6779 to look at the new patch set (#2). Change subject: env: add ReadV() API .. env: add ReadV() API Adds ReadV() metho

[kudu-CR] env: add ReadV() API

2017-05-02 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: env: add ReadV() API .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6779/1/src/kudu/util/env_posix.cc File src/kudu/util/env_posix.cc: Line 312:

[kudu-CR] env: add ReadV() API

2017-05-02 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: env: add ReadV() API .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/6779/1/src/kudu/fs/block_manager.h File src/kudu/fs/block_manager.h: Line 1

[kudu-CR] env: add ReadV() API

2017-05-02 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: env: add ReadV() API .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6779/1/src/kudu/util/env_posix.cc File src/kudu/util/env_posix.cc: Line 312:

[kudu-CR] env: add ReadV() API

2017-05-02 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: env: add ReadV() API .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/6779/1/src/kudu/fs/block_manager.h File src/kudu/fs/block_manager.h: PS1, Li

[kudu-CR] env: add ReadV() API

2017-05-02 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new change for review. http://gerrit.cloudera.org:8080/6779 Change subject: env: add ReadV() API .. env: add ReadV() API Adds ReadV() methods to RWFile and RandomAccessFile that allows reading data