[email protected] has posted comments on this change. ( http://gerrit.cloudera.org:8080/16200 )
Change subject: add setErrorCollectorSpace method to change ErrorCollector size, in case errors are dropped in manual flush mode ...................................................................... Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/16200/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/16200/1//COMMIT_MSG@7 PS1, Line 7: add setErrorCollectorSpace method to change ErrorCollector size, in case errors are dropped in manual flush mode > nit: line too long Done http://gerrit.cloudera.org:8080/#/c/16200/1/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java File java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java: PS1: > This approach seems a bit weird to me. Wouldn't it be better to allow speci I agree. Resizing errorCollector would be better. I have added resize method in ErrorCollector and some tests. Please have a look at "gerrit.cloudera.org/#/c/16201/". I'm sorry, I am not familiar with gerrit. I don't know why the amend has a different id and went into another page, although I git commit --amend. :d http://gerrit.cloudera.org:8080/#/c/16200/1/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java@229 PS1, Line 229: throw new IllegalArgumentException("Cannot change error collector size when error collecor is not empty"); > nit: line too long Done -- To view, visit http://gerrit.cloudera.org:8080/16200 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I8f04ff923fe53b28d1c54594948d660846331573 Gerrit-Change-Number: 16200 Gerrit-PatchSet: 1 Gerrit-Owner: Anonymous Coward <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Thu, 16 Jul 2020 07:32:52 +0000 Gerrit-HasComments: Yes
