[kudu-CR] KUDU-2230: java: sync client exception stack traces should make sense

2018-03-13 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9489 ) Change subject: KUDU-2230: java: sync client exception stack traces should make sense .. KUDU-2230: java: sync client exception

[kudu-CR] KUDU-2230: java: sync client exception stack traces should make sense

2018-03-13 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9489 ) Change subject: KUDU-2230: java: sync client exception stack traces should make sense .. Patch Set 2: > Actually I just remembered why I

[kudu-CR] KUDU-2230: java: sync client exception stack traces should make sense

2018-03-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9489 ) Change subject: KUDU-2230: java: sync client exception stack traces should make sense .. Patch Set 2: > Patch Set 2: > > > I was

[kudu-CR] KUDU-2230: java: sync client exception stack traces should make sense

2018-03-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9489 ) Change subject: KUDU-2230: java: sync client exception stack traces should make sense .. Patch Set 2: > I was thinking that "suppressed"

[kudu-CR] KUDU-2230: java: sync client exception stack traces should make sense

2018-03-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9489 ) Change subject: KUDU-2230: java: sync client exception stack traces should make sense .. Patch Set 2: > Patch Set 2: > > Why did you

[kudu-CR] KUDU-2230: java: sync client exception stack traces should make sense

2018-03-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9489 ) Change subject: KUDU-2230: java: sync client exception stack traces should make sense .. Patch Set 2: Why did you choose to model the

[kudu-CR] KUDU-2230: java: sync client exception stack traces should make sense

2018-03-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9489 ) Change subject: KUDU-2230: java: sync client exception stack traces should make sense .. Patch Set 2: Code-Review+2 (1 comment)

[kudu-CR] KUDU-2230: java: sync client exception stack traces should make sense

2018-03-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9489 ) Change subject: KUDU-2230: java: sync client exception stack traces should make sense .. Patch Set 2: Code-Review+1 (1 comment)

[kudu-CR] KUDU-2230: java: sync client exception stack traces should make sense

2018-03-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9489 ) Change subject: KUDU-2230: java: sync client exception stack traces should make sense .. Patch Set 2: (1 comment) This is a neat

[kudu-CR] KUDU-2230: java: sync client exception stack traces should make sense

2018-03-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9489 ) Change subject: KUDU-2230: java: sync client exception stack traces should make sense .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-2230: java: sync client exception stack traces should make sense

2018-03-12 Thread Todd Lipcon (Code Review)
Hello Alexey Serbin, Jean-Daniel Cryans, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9489 to look at the new patch set (#2). Change subject: KUDU-2230: java: sync client exception stack traces should make sense

[kudu-CR] KUDU-2230: java: sync client exception stack traces should make sense

2018-03-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9489 ) Change subject: KUDU-2230: java: sync client exception stack traces should make sense .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2230: java: sync client exception stack traces should make sense

2018-03-06 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9489 ) Change subject: KUDU-2230: java: sync client exception stack traces should make sense .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2230: java: sync client exception stack traces should make sense

2018-03-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9489 ) Change subject: KUDU-2230: java: sync client exception stack traces should make sense .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2230: java: sync client exception stack traces should make sense

2018-03-05 Thread Todd Lipcon (Code Review)
Hello Alexey Serbin, Jean-Daniel Cryans, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/9489 to review the following change. Change subject: KUDU-2230: java: sync client exception stack traces should make sense