Hello Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/21646
to look at the new patch set (#2).
Change subject: IMPALA-13240: Add gerrit comments for Thrift/FlatBuffers changes
......................................................................
IMPALA-13240: Add gerrit comments for Thrift/FlatBuffers changes
Adds gerrit comments for changes in Thrift/FlatBuffers files that could
break the communication between impalad and catalogd/statestore during
upgrade.
Basically, only new optional fields can be added in Thrift protocol. For
Flatbuffers schemas, we should only add new fields at the end of a table
definition.
Add a new option for critique-gerrit-review.py to specify the commit.
Tested with incompatible commits like 012996a06 and 65094a74f.
Change-Id: Ib35fafa50bfd38631312d22464df14d426f55346
---
M bin/jenkins/critique-gerrit-review.py
1 file changed, 136 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/46/21646/2
--
To view, visit http://gerrit.cloudera.org:8080/21646
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib35fafa50bfd38631312d22464df14d426f55346
Gerrit-Change-Number: 21646
Gerrit-PatchSet: 2
Gerrit-Owner: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
