[kudu-CR] [catalog manager] categorization of rw operation failures

2017-05-10 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: [catalog_manager] categorization of rw operation failures .. [catalog_manager] categorization of rw operation failures This changelist introduces

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-05-10 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 30: there you go :) -- To view, visit http://gerrit.cloudera.org:8080/6170 To

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-05-10 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 30: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6170 To

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-05-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 29: (6 comments) yep, it's been nits for the last 3 revisions :)

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-05-10 Thread Alexey Serbin (Code Review)
Hello David Ribeiro Alves, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#30). Change subject: [catalog_manager] categorization of rw operation failures

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-05-10 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 29: (6 comments) only nits, nearly there

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-05-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 29: Verified+1 Unrelated flake tracked by:

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-05-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 28: (21 comments) http://gerrit.cloudera.org:8080/#/c/6170/28//COMMIT_MSG Commit

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-05-09 Thread Alexey Serbin (Code Review)
Hello David Ribeiro Alves, Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#29). Change subject: [catalog_manager] categorization of rw operation failures

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-05-09 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 28: (21 comments) http://gerrit.cloudera.org:8080/#/c/6170/28//COMMIT_MSG Commit

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-05-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 26: (5 comments)

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-05-06 Thread Alexey Serbin (Code Review)
Hello David Ribeiro Alves, Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#28). Change subject: [catalog_manager] categorization of rw operation failures

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-05-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 26: (23 comments)

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-05-06 Thread Alexey Serbin (Code Review)
Hello David Ribeiro Alves, Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#27). Change subject: [catalog_manager] categorization of rw operation failures

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-05-04 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 26: (25 comments)

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 26: > Build Successful > > http://104.196.14.100/job/kudu-gerrit/6937/ : SUCCESS 1024

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 26: > Uploaded patch set 26. The race has gone:

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-09 Thread Alexey Serbin (Code Review)
Hello David Ribeiro Alves, Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#26). Change subject: [catalog_manager] categorization of rw operation failures

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-09 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 25: you're right, the logs show that TSK skips a number. e.g. 23, 24, 26, 25 --

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 25: > I chased the bug. It's the following. Say node A is at term 10 and > is leader

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 25: > Sorry for the rebase. Did it to post a WIP of my debugging code: >

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-09 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 24: Sorry for the rebase. Did it to post a WIP of my debugging code:

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 24: > > > Given that we're still chasing strange test failures on this, > and > > >

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-09 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 24: I chased the bug. It's the following. Say node A is at term 10 and is leader

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 24: > > Given that we're still chasing strange test failures on this, and > > it's on a

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 24: > Given that we're still chasing strange test failures on this, and > it's on a

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 24: Given that we're still chasing strange test failures on this, and it's on a pretty

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 24: > Uploaded patch set 24. The test results at:

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 22: (1 comment)

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-08 Thread Alexey Serbin (Code Review)
Hello David Ribeiro Alves, Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#23). Change subject: [catalog_manager] categorization of rw operation failures

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-08 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 22: when you do loop in dist tests or something, please post the command/build

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-08 Thread Alexey Serbin (Code Review)
Hello David Ribeiro Alves, Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#22). Change subject: [catalog_manager] categorization of rw operation failures

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 21: > could you loop the test a bit? Sure -- that's what I did yesterday. But I was

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-08 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 21: Code-Review+1 could you loop the test a bit? -- To view, visit

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-08 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#21). Change subject: [catalog_manager] categorization of rw operation failures

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-08 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#20). Change subject: [catalog_manager] categorization of rw operation failures

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 18: (1 comment)

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-08 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#19). Change subject: [catalog_manager] categorization of rw operation failures

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 18: (9 comments)

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-08 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 18: (4 comments)

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-08 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 10: (5 comments)

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-08 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#18). Change subject: [catalog_manager] categorization of rw operation failures ..

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 17: Verified+1 Flake: MasterFailoverTest.TestCreateTableSync -- To view, visit

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-07 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#17). Change subject: [catalog_manager] categorization of rw operation failures

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-07 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#16). Change subject: [catalog_manager] categorization of rw operation failures

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-07 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#15). Change subject: [catalog_manager] categorization of rw operation failures

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 13: (1 comment)

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 13: (1 comment)

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 13: (4 comments)

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-07 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#14). Change subject: [catalog_manager] categorization of rw operation failures

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 13: (1 comment)

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 13: (3 comments)

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-07 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#13). Change subject: [catalog_manager] categorization of rw operation failures

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 11: (2 comments) Added ./bin/catalog_manager_tsk-itest

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-07 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#12). Change subject: [catalog_manager] categorization of rw operation failures

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-06 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 11: (1 comment) I think the previous rev demonstrated that this is lacking

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 10: (10 comments)

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-06 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#11). Change subject: [catalog_manager] categorization of rw operation failures

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 10: (10 comments)

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-06 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 10: (3 comments) http://gerrit.cloudera.org:8080/#/c/6170/10//COMMIT_MSG Commit

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 9: (1 comment)

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-03 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#10). Change subject: [catalog_manager] categorization of rw operation failures ..

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-03 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 9: Code-Review+1 (1 comment)

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 9: Verified+1 Flake: org.apache.kudu.spark.kudu.DefaultSourceTest.Test SparkSQL

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 8: (10 comments) http://gerrit.cloudera.org:8080/#/c/6170/8//COMMIT_MSG Commit Message:

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-03 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#9). Change subject: [catalog_manager] categorization of rw operation failures ..

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-03 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 8: (10 comments) http://gerrit.cloudera.org:8080/#/c/6170/8//COMMIT_MSG Commit Message:

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [catalog_manager] categorization of rw operation failures .. Patch Set 8: Verified+1 Looks like a couple of flakes:

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-03 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#8). Change subject: [catalog_manager] categorization of rw operation failures

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-03 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#7). Change subject: [catalog_manager] categorization of rw operation failures

[kudu-CR] [catalog manager] categorization of rw operation failures

2017-03-02 Thread Alexey Serbin (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6170 to look at the new patch set (#6). Change subject: [catalog_manager] categorization of rw operation failures