Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/18075 )
Change subject: [client] KUDU-2623 Expose table as public in WriteOp ...................................................................... Patch Set 3: > > Patch Set 3: Code-Review+2 > > > > LGTM. Please make sure that somewhat similar approach works for > the Java client as well -- the idea is that ideally this have to > almost 'mirror' the approach in Java client. > > Thank you! > > I did check the Java client, there the table object is publicly > accessible, e.g.: > RowErrorsAndOverflowStatus reos = session.getPendingErrors(); > reos.getRowErrors()[0].getOperation().getTable().getName(); > > I'm still working on confirming this on the python client. Ah, cool: thank you for clarifying on this! -- To view, visit http://gerrit.cloudera.org:8080/18075 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Idbfff6e931304cce7826bc272811155f57a5c4e5 Gerrit-Change-Number: 18075 Gerrit-PatchSet: 3 Gerrit-Owner: Zoltan Chovan <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan <[email protected]> Gerrit-Comment-Date: Fri, 10 Dec 2021 17:51:52 +0000 Gerrit-HasComments: No
