Lars Volker has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/12543


Change subject: IMPALA-8234: Fix ordering of Thrift enum, add warning
......................................................................

IMPALA-8234: Fix ordering of Thrift enum, add warning

IMPALA-7694 added a field in the middle of the Metrics.TUnit enum, which
broke backwards compatibility with profiles that had been written by
older versions of Impala. This change fixes the ordering by moving the
field to the end of the enum.

Additionally, it adds a warning to the top of all Thrift files that are
part of the binary profile format, and an note of caution to the main
definition in RuntimeProfile.thrift.

Change-Id: If215f16a636008757ceb439edbd6900a1be88c59
---
M common/thrift/ExecStats.thrift
M common/thrift/Metrics.thrift
M common/thrift/RuntimeProfile.thrift
M common/thrift/Status.thrift
M common/thrift/Types.thrift
5 files changed, 32 insertions(+), 7 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/43/12543/1
--
To view, visit http://gerrit.cloudera.org:8080/12543
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: If215f16a636008757ceb439edbd6900a1be88c59
Gerrit-Change-Number: 12543
Gerrit-PatchSet: 1
Gerrit-Owner: Lars Volker <l...@cloudera.com>

Reply via email to