Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8151 )
Change subject: IMPALA-5307: Part 3: remove TODO from RCFile ...................................................................... IMPALA-5307: Part 3: remove TODO from RCFile Our RCFile implementations already copies out data (it sets set_contains_tuple_data to false). Remove a TODO that suggests undoing this. The current implementation is suboptimal but improving RCFile performance is not a priority. Change-Id: I594bb246cab64e15de750114890881a2ad9f504d Reviewed-on: http://gerrit.cloudera.org:8080/8151 Reviewed-by: Tim Armstrong <[email protected]> Tested-by: Impala Public Jenkins --- M be/src/exec/hdfs-rcfile-scanner.cc 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Tim Armstrong: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/8151 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I594bb246cab64e15de750114890881a2ad9f504d Gerrit-Change-Number: 8151 Gerrit-PatchSet: 3 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Tim Armstrong <[email protected]>
