Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/13908 )
Change subject: Deprecated TabletLocationsPB.ReplicaPB message ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/13908/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/13908/1//COMMIT_MSG@9 PS1, Line 9: According to the comments of https://gerrit.cloudera.org/#/c/13875/, the hrm, in those comments, it claimed that ReplicaPB was deprecated as of 0.5.0, but I'm not sure where that came from. The new InternedReplicaPB was added in 586e957f76a547340f2ab93a7eebc3f116ff825e which was only a few months ago. I think removing ReplicaPB would break wire compatibility with 1.9 and below? -- To view, visit http://gerrit.cloudera.org:8080/13908 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I18272274f07d5ae8e9f6b9572f9900aa8df27bef Gerrit-Change-Number: 13908 Gerrit-PatchSet: 1 Gerrit-Owner: Yao Xu <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Wed, 24 Jul 2019 07:19:38 +0000 Gerrit-HasComments: Yes
