Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/16567 )
Change subject: IMPALA-9485: Enable file handle cache for EC files ...................................................................... Patch Set 2: Code-Review+2 This makes sense to me. It is nice to remove all the plumbing for the EC value. The only question is whether we will ever want that value for something else (stats, etc). I guess we can always revive this plumbing if we need it down the line. -- To view, visit http://gerrit.cloudera.org:8080/16567 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ieb455eeed02a229a4559d3972dfdac7df32cdb99 Gerrit-Change-Number: 16567 Gerrit-PatchSet: 2 Gerrit-Owner: Sahil Takiar <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Sahil Takiar <[email protected]> Gerrit-Comment-Date: Fri, 09 Oct 2020 01:21:06 +0000 Gerrit-HasComments: No
