[kudu-CR] KUDU-1755 Part 2: Improve tablet on-disk size metric

2017-09-18 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1755 Part 2: Improve tablet on-disk size metric .. Patch Set 19: (1 comment)

[kudu-CR] standard notation for fixed-size integer type names

2017-09-18 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/8099 to look at the new patch set (#2). Change subject: standard notation for fixed-size integer type names

[kudu-CR] standard notation for fixed-size integer type names

2017-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: standard notation for fixed-size integer type names .. Patch Set 1: Code-Review+2 Looks OK to me, though I'm curious whether Todd thinks we should stay with the

[kudu-CR] catalog manager: make ScopedTabletInfoCommitter generic

2017-09-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: catalog_manager: make ScopedTabletInfoCommitter generic .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/8089/2/src/kudu/util/cow_object.h File

[kudu-CR] catalog manager: make ScopedTabletInfoCommitter generic

2017-09-18 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: catalog_manager: make ScopedTabletInfoCommitter generic .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/8089/2/src/kudu/util/cow_object.h File

[kudu-CR] Link to troubleshooting docs for two common problems

2017-09-18 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: Link to troubleshooting docs for two common problems .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/8093/3/docs/troubleshooting.adoc File

[kudu-CR] KUDU-2055 [part 1]: Coalesce hole punching when deleting groups of blocks

2017-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-2055 [part 1]: Coalesce hole punching when deleting groups of blocks .. Patch Set 6: (9 comments) Here's my first pass. I'll do a deeper pass after you've

[kudu-CR] Link to troubleshooting docs for two common problems

2017-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Link to troubleshooting docs for two common problems .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/8093/2/docs/troubleshooting.adoc File

[kudu-CR] Link to troubleshooting docs for two common problems

2017-09-18 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Link to troubleshooting docs for two common problems .. Patch Set 3: Code-Review+1 lgtm adar's nits -- To view, visit

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

2017-09-18 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1807 (part 3): remove GetTableSchema.create_table_done .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8045 To

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

2017-09-18 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1807 (part 3): remove GetTableSchema.create_table_done .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8045 To unsubscribe,

[kudu-CR] data dirs: fix a LOG statement

2017-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: data_dirs: fix a LOG statement .. data_dirs: fix a LOG statement This is currently logging like so: No directories available to add to

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

2017-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: KUDU-1807 (part 1): deprecate GetTableSchema.create_table_done .. KUDU-1807 (part 1): deprecate GetTableSchema.create_table_done Improving

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

2017-09-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1807 (part 2): ban GetTableSchema for table createdness in clients .. Patch Set 5: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2135 (part 2): don't use previously failed disks

2017-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-2135 (part 2): don't use previously failed disks .. Patch Set 30: (2 comments) Not many new comments from me since I already reviewed this a while ago, and

[kudu-CR] Link to troubleshooting docs for two common problems

2017-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Link to troubleshooting docs for two common problems .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/8093/3/src/kudu/fs/fs_manager.cc File

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

2017-09-18 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-2055 [part 2]: Add util to construct sorted disjoint interval list .. Patch Set 1: (1 comment)

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

2017-09-18 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-2055 [part 2]: Add util to construct sorted disjoint interval list .. Patch Set 2: (5 comments)

[kudu-CR] [docs] fixup releasing git instructions

2017-09-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [docs] fixup releasing git instructions .. [docs] fixup releasing git instructions Change-Id: I83f98350ddcaf062477db1ee0b90f1811f87524b

[kudu-CR] [docs] fixup releasing git instructions

2017-09-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [docs] fixup releasing git instructions .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7876 To unsubscribe, visit

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

2017-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-2135 (part 1): add persistent disk states .. Patch Set 7: (14 comments) I think Todd should review this.

[kudu-CR] Link to troubleshooting docs for two common problems

2017-09-18 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: Link to troubleshooting docs for two common problems .. Patch Set 2: (8 comments) http://gerrit.cloudera.org:8080/#/c/8093/2/docs/troubleshooting.adoc File

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

2017-09-18 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: KUDU-2055 [part 2]: Add util to construct sorted disjoint interval list .. Patch Set 2: (21 comments) http://gerrit.cloudera.org:8080/#/c/8041/1//COMMIT_MSG Commit

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

2017-09-18 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8041 to look at the new patch set (#2). Change subject: KUDU-2055 [part 2]: Add util to construct sorted disjoint interval list

[kudu-CR] KUDU-1755 Part 2: Improve tablet on-disk size metric

2017-09-18 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1755 Part 2: Improve tablet on-disk size metric .. Patch Set 19: (1 comment)

[kudu-CR] Link to troubleshooting docs for two common problems

2017-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Link to troubleshooting docs for two common problems .. Patch Set 2: (5 comments) http://gerrit.cloudera.org:8080/#/c/8093/2/docs/troubleshooting.adoc File

[kudu-CR] KUDU-1755 Part 2: Improve tablet on-disk size metric

2017-09-18 Thread Will Berkeley (Code Review)
Hello Adar Dembo, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6968 to look at the new patch set (#21). Change subject: KUDU-1755 Part 2: Improve tablet on-disk size metric

[kudu-CR] KUDU-1755 Part 2: Improve tablet on-disk size metric

2017-09-18 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-1755 Part 2: Improve tablet on-disk size metric .. Patch Set 21: Something mysterious happened and so 19 + 1 = 21. -- To view, visit

[kudu-CR] KUDU-1755 Part 2: Improve tablet on-disk size metric

2017-09-18 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-1755 Part 2: Improve tablet on-disk size metric .. Patch Set 19: (3 comments) http://gerrit.cloudera.org:8080/#/c/6968/19/src/kudu/consensus/log.h File

[kudu-CR] Link to troubleshooting docs for two common problems

2017-09-18 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Link to troubleshooting docs for two common problems .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/8093/2/docs/troubleshooting.adoc

[kudu-CR] [iwyu] update std::move

2017-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [iwyu] update std::move .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/8088/1/build-support/iwyu/mappings/libstdcpp.imp File

[kudu-CR] Bump protobuf to 3.4.1

2017-09-18 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: Bump protobuf to 3.4.1 .. Bump protobuf to 3.4.1 This allows us to drop the 'experimental' flag for move constructors introduced in 1632d16aa5993, since

[kudu-CR] c++ client: try harder to pass table IDs into RPCs that can accept them

2017-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: c++ client: try harder to pass table IDs into RPCs that can accept them .. c++ client: try harder to pass table IDs into RPCs that can accept them Some

[kudu-CR] [iwyu] update std::move

2017-09-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [iwyu] update std::move .. Patch Set 1: (26 comments) Thank you for the review.

[kudu-CR] Bump protobuf to 3.4.1

2017-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Bump protobuf to 3.4.1 .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8086 To unsubscribe, visit

[kudu-CR] Link to troubleshooting docs for two common problems

2017-09-18 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8093 to look at the new patch set (#2). Change subject: Link to troubleshooting docs for two common problems .. Link to

[kudu-CR] [docs] fixup releasing git instructions

2017-09-18 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [docs] fixup releasing git instructions .. Patch Set 3: This is ready to review -- To view, visit http://gerrit.cloudera.org:8080/7876 To unsubscribe, visit

[kudu-CR] kudu-spark: change default master address to local fqdn

2017-09-18 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: kudu-spark: change default master address to local fqdn .. Patch Set 2: (1 comment)

[kudu-CR] WIP: Import ArrayView type from WebRTC

2017-09-18 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: WIP: Import ArrayView type from WebRTC .. Patch Set 1: woops. duplicate. -- To view, visit http://gerrit.cloudera.org:8080/8095 To unsubscribe, visit

[kudu-CR] WIP: Import ArrayView type from WebRTC

2017-09-18 Thread Dan Burkert (Code Review)
Dan Burkert has abandoned this change. Change subject: WIP: Import ArrayView type from WebRTC .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/8095 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

[kudu-CR] WIP: Import ArrayView type from WebRTC

2017-09-18 Thread Dan Burkert (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8095 to review the following change. Change subject: WIP: Import ArrayView type from WebRTC .. WIP: Import ArrayView type

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

2017-09-18 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: docs: clarify steps for changing master from multi-master deployment .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8032 To

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

2017-09-18 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: docs: clarify steps for changing master from multi-master deployment .. docs: clarify steps for changing master from multi-master deployment The current

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

2017-09-18 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: docs: clarify steps for changing master from multi-master deployment .. Patch Set 3: (1 comment)

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

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

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

2017-09-18 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Kudu Consistency Blog Post Pt1 .. Patch Set 14: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7019 To unsubscribe, visit

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

2017-09-18 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: Kudu Consistency Blog Post Pt1 .. Kudu Consistency Blog Post Pt1 This is the first part of multi-part blog post series about consistency in Kudu.

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

2017-09-18 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: Kudu Consistency Blog Post Pt1 .. Patch Set 14: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7019 To unsubscribe, visit

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

2017-09-18 Thread David Ribeiro Alves (Code Review)
Hello Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7019 to look at the new patch set (#14). Change subject: Kudu Consistency Blog Post Pt1 .. Kudu Consistency Blog Post

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

2017-09-18 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Kudu Consistency Blog Post Pt1 .. Patch Set 13: (1 comment) http://gerrit.cloudera.org:8080/#/c/7019/13/_posts/2017-09-18-kudu-consistency-pt1.md File

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

2017-09-18 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: Kudu Consistency Blog Post Pt1 .. Patch Set 13: (1 comment) One thing then I'm +2

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

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

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

2017-09-18 Thread David Ribeiro Alves (Code Review)
Hello Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7019 to look at the new patch set (#13). Change subject: Kudu Consistency Blog Post Pt1 .. Kudu Consistency Blog Post

[kudu-CR] Bump protobuf to 3.4.1

2017-09-18 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Bump protobuf to 3.4.1 .. Patch Set 2: Yeah this seems to be protobuf's MO: https://github.com/google/protobuf/issues/3602 -- To view, visit

[kudu-CR] Bump protobuf to 3.4.1

2017-09-18 Thread Dan Burkert (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8086 to look at the new patch set (#3). Change subject: Bump protobuf to 3.4.1 .. Bump protobuf to 3.4.1

[kudu-CR] Bump protobuf to 3.4.1

2017-09-18 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Bump protobuf to 3.4.1 .. Patch Set 2: https://github.com/google/protobuf/issues/3656 I'm going to change it back to 3.4.0 in the meantime, since I suspect it

[kudu-CR] c++ client: try harder to pass table IDs into RPCs that can accept them

2017-09-18 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: c++ client: try harder to pass table IDs into RPCs that can accept them .. Patch Set 3: (1 comment)

[kudu-CR] Link to troubleshooting docs for two common problems

2017-09-18 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: Link to troubleshooting docs for two common problems .. Patch Set 1: Not really for review just yet- will be IWYU things to fix and I want a unit test run

[kudu-CR] Link to troubleshooting docs for two common problems

2017-09-18 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded a new change for review. http://gerrit.cloudera.org:8080/8093 Change subject: Link to troubleshooting docs for two common problems .. Link to troubleshooting docs for two common problems >From

[kudu-CR] WIP [raft consensus-itest] fix flake in TestSlowLeader

2017-09-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: WIP [raft_consensus-itest] fix flake in TestSlowLeader .. Patch Set 2: > can you explain a little better the exact scenario that happens? > could you cause it

[kudu-CR] data dirs: fix a LOG statement

2017-09-18 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: data_dirs: fix a LOG statement .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8091 To unsubscribe, visit

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

2017-09-18 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1807 (part 3): remove GetTableSchema.create_table_done .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8045 To

[kudu-CR] c++ client: try harder to pass table IDs into RPCs that can accept them

2017-09-18 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: c++ client: try harder to pass table IDs into RPCs that can accept them .. Patch Set 4: Code-Review+2 -- To view, visit

[kudu-CR] c++ client: try harder to pass table IDs into RPCs that can accept them

2017-09-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: c++ client: try harder to pass table IDs into RPCs that can accept them .. Patch Set 4: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/8066