Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11026 )
Change subject: IMPALA-7340. Only send necessary fields in THdfsPartition ...................................................................... IMPALA-7340. Only send necessary fields in THdfsPartition The THdfsPartition Thrift struct is used both for partition metadata (sent from catalogd to coordinator) and for descriptors (sent from coordinator frontend to backends). In the case of the descriptor, not all fields are actually used. This patch cleans up the Thrift struct definition to be more clear which fields are used where, and changes the serialization code to only fill in the necessary fields. Change-Id: I97e8402efdfdeea06463bb71a40ebb6abd1f11f0 Reviewed-on: http://gerrit.cloudera.org:8080/11026 Tested-by: Impala Public Jenkins <[email protected]> Reviewed-by: Todd Lipcon <[email protected]> --- M common/thrift/CatalogObjects.thrift M fe/src/main/java/org/apache/impala/catalog/CatalogObject.java M fe/src/main/java/org/apache/impala/catalog/FeCatalogUtils.java M fe/src/main/java/org/apache/impala/catalog/HdfsTable.java M fe/src/main/java/org/apache/impala/catalog/local/LocalFsTable.java 5 files changed, 90 insertions(+), 43 deletions(-) Approvals: Impala Public Jenkins: Verified Todd Lipcon: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/11026 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I97e8402efdfdeea06463bb71a40ebb6abd1f11f0 Gerrit-Change-Number: 11026 Gerrit-PatchSet: 4 Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Tianyi Wang <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Reviewer: Vuk Ercegovac <[email protected]>
