[kudu-CR] Add tablet state summary metrics

2017-09-12 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: Add tablet state summary metrics .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/7980/7/src/kudu/tserver/ts_tablet_manager_metrics.h File

[kudu-CR] Add tablet state summary metrics

2017-09-12 Thread Will Berkeley (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7980 to look at the new patch set (#8). Change subject: Add tablet state summary metrics .. Add tablet

[kudu-CR] KUDU-2135 (part 2): check integrity of new on-disk format

2017-09-12 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7270 to look at the new patch set (#28). Change subject: KUDU-2135 (part 2): check integrity of new on-disk format ..

[kudu-CR] KUDU-2135 (part 1): add persistent disk states

2017-09-12 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8048 to look at the new patch set (#2). Change subject: KUDU-2135 (part 1): add persistent disk states .. KUDU-2135

[kudu-CR] docs: clarify steps for removing master from multi-master deployment

2017-09-12 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: docs: clarify steps for removing master from multi-master deployment .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-2125: Tablet copy client does not retry on failures

2017-09-12 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-2125: Tablet copy client does not retry on failures .. Patch Set 10: (5 comments) Just one thing I'm wondering about, in the tablet copy client.

[kudu-CR](gh-pages) Kudu Consistency Blog Post Pt1

2017-09-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Kudu Consistency Blog Post Pt1 .. Patch Set 12: (5 comments) http://gerrit.cloudera.org:8080/#/c/7019/12/_posts/2017-08-21-kudu-consistency-pt1.md File

[kudu-CR] KUDU-2135 (part 1): add persistent disk states

2017-09-12 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded a new change for review. http://gerrit.cloudera.org:8080/8048 Change subject: KUDU-2135 (part 1): add persistent disk states .. KUDU-2135 (part 1): add persistent disk states This patch introduces a

[kudu-CR] KUDU-2124. Don't hold session lock while initializing a TabletCopySession

2017-09-12 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-2124. Don't hold session lock while initializing a TabletCopySession .. Patch Set 8: -Code-Review (1 comment)

[kudu-CR] KUDU-2124. Don't hold session lock while initializing a TabletCopySession

2017-09-12 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-2124. Don't hold session lock while initializing a TabletCopySession .. Patch Set 8: Code-Review+2 (2 comments)

[kudu-CR] Add tablet state summary metrics

2017-09-12 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add tablet state summary metrics .. Patch Set 7: (1 comment) looks good, just one nit on something that changed since the last review

[kudu-CR] KUDU-1788. Increase Raft RPC timeout to 30sec to avoid fruitless retries.

2017-09-12 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1788. Increase Raft RPC timeout to 30sec to avoid fruitless retries. .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-1807 (part 3): remove GetTableSchema.create table done

2017-09-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1807 (part 3): remove GetTableSchema.create_table_done .. Patch Set 1: Verified+1 NTP synchronization error in one of the dist-test slaves, apparently. --

[kudu-CR] KUDU-1807 (part 1): deprecate GetTableSchema.create table done

2017-09-12 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8027 to look at the new patch set (#2). Change subject: KUDU-1807 (part 1): deprecate GetTableSchema.create_table_done

[kudu-CR] KUDU-1807 (part 2): ban GetTableSchema for table createdness in clients

2017-09-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1807 (part 2): ban GetTableSchema for table createdness in clients .. Patch Set 1: (9 comments) http://gerrit.cloudera.org:8080/#/c/8026/1//COMMIT_MSG

[kudu-CR] KUDU-1807 (part 3): remove GetTableSchema.create table done

2017-09-12 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8045 to review the following change. Change subject: KUDU-1807 (part 3): remove GetTableSchema.create_table_done

[kudu-CR] KUDU-1807 (part 2): ban GetTableSchema for table createdness in clients

2017-09-12 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8026 to look at the new patch set (#2). Change subject: KUDU-1807 (part 2): ban GetTableSchema for table createdness in clients

[kudu-CR] KUDU-2055 [part 2]: Add util to construct sorted disjoint interval

2017-09-12 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new change for review. http://gerrit.cloudera.org:8080/8041 Change subject: KUDU-2055 [part 2]: Add util to construct sorted disjoint interval .. KUDU-2055 [part 2]: Add util to construct sorted

[kudu-CR](gh-pages) Kudu Consistency Blog Post Pt1

2017-09-12 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: Kudu Consistency Blog Post Pt1 .. Patch Set 12: (3 comments) Just a couple of things + Andrew's nits

[kudu-CR] docs: clarify steps for removing master from multi-master deployment

2017-09-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: docs: clarify steps for removing master from multi-master deployment .. Patch Set 2: (4 comments)

[kudu-CR] KUDU-2124. Don't hold session lock while initializing a TabletCopySession

2017-09-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-2124. Don't hold session lock while initializing a TabletCopySession .. Patch Set 8: Code-Review+1 Mike should review this. -- To view, visit

[kudu-CR] KUDU-2125: Tablet copy client does not retry on failures

2017-09-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-2125: Tablet copy client does not retry on failures .. Patch Set 10: Code-Review+1 Mike should review this. -- To view, visit

[kudu-CR] KUDU-2125: Tablet copy client does not retry on failures

2017-09-12 Thread Dan Burkert (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8016 to look at the new patch set (#10). Change subject: KUDU-2125: Tablet copy client does not retry on failures

[kudu-CR] KUDU-2125: Tablet copy client does not retry on failures

2017-09-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-2125: Tablet copy client does not retry on failures .. Patch Set 8: (1 comment)

[kudu-CR] docs: clarify steps for removing master from multi-master deployment

2017-09-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: docs: clarify steps for removing master from multi-master deployment .. Patch Set 2: Code-Review+1 (2 comments)

[kudu-CR] KUDU-2125: Tablet copy client does not retry on failures

2017-09-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-2125: Tablet copy client does not retry on failures .. Patch Set 8: Code-Review+1 (1 comment)

[kudu-CR](gh-pages) Update website for 1.5.0 release

2017-09-12 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: Update website for 1.5.0 release .. Update website for 1.5.0 release Change-Id: Id8885cd6ecfb904e95370e566a32a579f6c85a4a Reviewed-on:

[kudu-CR](gh-pages) Update website for 1.5.0 release

2017-09-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Update website for 1.5.0 release .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/8013 To unsubscribe, visit

[kudu-CR](gh-pages) Update website for 1.5.0 release

2017-09-12 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Update website for 1.5.0 release .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8013 To unsubscribe, visit

[kudu-CR](gh-pages) Update website for 1.5.0 release

2017-09-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Update website for 1.5.0 release .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/8013/1/releases/index.md File releases/index.md: Line 12: *

[kudu-CR] KUDU-2125: Tablet copy client does not retry on failures

2017-09-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-2125: Tablet copy client does not retry on failures .. Patch Set 8: (1 comment)

[kudu-CR] KUDU-2125: Tablet copy client does not retry on failures

2017-09-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-2125: Tablet copy client does not retry on failures .. Patch Set 8: (1 comment)

[kudu-CR] KUDU-1788. Increase Raft RPC timeout to 30sec to avoid fruitless retries.

2017-09-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1788. Increase Raft RPC timeout to 30sec to avoid fruitless retries. .. Patch Set 1: does this have any nasty interaction with failure detection for

[kudu-CR](gh-pages) Update website for 1.5.0 release

2017-09-12 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded a new patch set (#2). Change subject: Update website for 1.5.0 release .. Update website for 1.5.0 release Change-Id: Id8885cd6ecfb904e95370e566a32a579f6c85a4a --- M apidocs/allclasses-frame.html M

[kudu-CR] KUDU-1788. Increase Raft RPC timeout to 30sec to avoid fruitless retries.

2017-09-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1788. Increase Raft RPC timeout to 30sec to avoid fruitless retries. .. Patch Set 1: (1 comment)