Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/14107 )
Change subject: KUDU-2921: Exposing the table statistics to spark relation. ...................................................................... Patch Set 1: > Patch Set 1: > > Did you consider adding the statistics to the existing GetTableLocations RPC > response instead of creating a brand new RPC? Query planners and job drivers > already use it to open tables, so it'd mean one less round trip to get the > information. If doing this, let's make sure to make it optional, since the statistics could become larger later (eg with things like histograms, NDVs per column, etc) -- To view, visit http://gerrit.cloudera.org:8080/14107 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I7742a76708f989b0ccc8ba417f3390013e260175 Gerrit-Change-Number: 14107 Gerrit-PatchSet: 1 Gerrit-Owner: ZhangYao <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Tue, 20 Aug 2019 17:28:53 +0000 Gerrit-HasComments: No
