[kudu-CR] feat: add task interruption checking for RowIterator

2017-09-03 Thread caiconghui (Code Review)
caiconghui has posted comments on this change. Change subject: feat: add task interruption checking for RowIterator .. Patch Set 4: @Kudu Jenkins, The failure seems that the exception below is not caused by the new feature?

[kudu-CR] feat: add task interruption checking for RowIterator

2017-09-01 Thread caiconghui (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7753 to look at the new patch set (#4). Change subject: feat: add task interruption checking for RowIterator .. feat:

[kudu-CR] feat: add task interruption checking for RowIterator

2017-08-31 Thread caiconghui (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7753 to look at the new patch set (#3). Change subject: feat: add task interruption checking for RowIterator .. feat:

[kudu-CR] feat: add the wrapper for the Iterator by using InterruptibleIterator

2017-08-30 Thread caiconghui (Code Review)
caiconghui has posted comments on this change. Change subject: feat: add the wrapper for the Iterator by using InterruptibleIterator .. Patch Set 2: @Dan Burkert Yeah, I think throw TaskKilledException("KuduRDD task

[kudu-CR] feat: add the wrapper for the Iterator by using InterruptibleIterator

2017-08-29 Thread caiconghui (Code Review)
caiconghui has posted comments on this change. Change subject: feat: add the wrapper for the Iterator by using InterruptibleIterator .. Patch Set 2: @Dan Burkert I cannot find the appropriate api for canceling task or killing

[kudu-CR] add outputmetric collecting for the kudu data writing

2017-08-23 Thread caiconghui (Code Review)
caiconghui has posted comments on this change. Change subject: add outputmetric collecting for the kudu data writing .. Patch Set 1: @Kudu Jenkins, ok.I try to find the public api for collecting from spark, but failed. The

[kudu-CR] feat: add the wrapper for the Iterator by using InterruptibleIterator

2017-08-23 Thread caiconghui (Code Review)
caiconghui has posted comments on this change. Change subject: feat: add the wrapper for the Iterator by using InterruptibleIterator .. Patch Set 2: @Kudu Jenkins in my practice, the InterruptibleIterator dosen't reduce a lot

[kudu-CR] feat: add the wrapper for the Iterator by using InterruptibleIterator

2017-08-23 Thread caiconghui (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7753 to look at the new patch set (#2). Change subject: feat: add the wrapper for the Iterator by using InterruptibleIterator

[kudu-CR] add the wrapper for the Iterator[Row] of KuduRDD by using Spark InterruptibleIterator[Row]

2017-08-21 Thread caiconghui (Code Review)
caiconghui has posted comments on this change. Change subject: add the wrapper for the Iterator[Row] of KuduRDD by using Spark InterruptibleIterator[Row] .. Patch Set 1: @Kudu Jenkins what't wrong with the new feature. it

[kudu-CR] add outputmetric collecting for the kudu data writing

2017-08-21 Thread caiconghui (Code Review)
caiconghui has uploaded a new change for review. http://gerrit.cloudera.org:8080/7754 Change subject: add outputmetric collecting for the kudu data writing .. add outputmetric collecting for the kudu data writing once we