Todd Lipcon has posted comments on this change.

Change subject: Avoid unnecessary vector<Slice> allocations for 
ReadV/WriteV-like APIs
......................................................................


Patch Set 4:

I'm going to go ahead and push this rather than wait for the LEGAL jira to be 
resolved. So long as it's approved prior to our next release we should be fine, 
and I feel relatively confident in it. If for some reason it's not approved we 
can work on swapping out ArrayView for our own implementation or gsl::span

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I4eab29dad2e16cc5fce724d3bdd173f3a60cb266
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: No

Reply via email to