[kudu-CR] [tools] ksck improvements [3/n]: master consensus checks

2018-04-10 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/9883 ) Change subject: [tools] ksck improvements [3/n]: master consensus checks .. Patch Set 5: -Verified (11 comments)

[kudu-CR] [tools] ksck improvements [3/n]: master consensus checks

2018-04-10 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/9883 ) Change subject: [tools] ksck improvements [3/n]: master consensus checks .. Patch Set 6: (2 comments)

[kudu-CR] [tools] ksck improvements [4/n]: KUDU-2271 and KUDU-2310

2018-04-10 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/9912 ) Change subject: [tools] ksck improvements [4/n]: KUDU-2271 and KUDU-2310 .. Patch Set 2: -Verified (1 comment)

[kudu-CR] [tools] ksck improvements [3/n]: master consensus checks

2018-04-10 Thread Will Berkeley (Code Review)
Hello Mike Percy, Attila Bukor, Andrew Wong, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9883 to look at the new patch set (#6). Change subject: [tools] ksck improvements [3/n]: master consensus checks

[kudu-CR] [tools] ksck improvements [3/n]: master consensus checks

2018-04-10 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/9883 ) Change subject: [tools] ksck improvements [3/n]: master consensus checks .. Patch Set 6: (1 comment)

[kudu-CR] KUDU-2191 (9/n): HMS Catalog should short-circuit no-op alter and create table ops

2018-04-10 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9965 to look at the new patch set (#2). Change subject: KUDU-2191 (9/n): HMS Catalog should short-circuit no-op alter and create table ops

[kudu-CR] [docs] Add docs on full data dirs

2018-04-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9967 ) Change subject: [docs] Add docs on full data dirs .. Patch Set 1: (2 comments)

[kudu-CR] [tools] ksck improvements [3/n]: master consensus checks

2018-04-10 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/9883 ) Change subject: [tools] ksck improvements [3/n]: master consensus checks .. Patch Set 7: Code-Review+1 -- To view, visit

[kudu-CR] [tools] ksck improvements [3/n]: master consensus checks

2018-04-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9883 ) Change subject: [tools] ksck improvements [3/n]: master consensus checks .. Patch Set 7: Code-Review+1 (1 comment) Also some IWYU

[kudu-CR] releasing: update gerrit update instructions

2018-04-10 Thread Mike Percy (Code Review)
Mike Percy has abandoned this change. ( http://gerrit.cloudera.org:8080/9950 ) Change subject: releasing: update gerrit update instructions .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/9950 To unsubscribe,

[kudu-CR] releasing: update gerrit update instructions

2018-04-10 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9950 ) Change subject: releasing: update gerrit update instructions .. Patch Set 1: I think your account was set up more correctly than mine.

[kudu-CR] pstack watcher: blacklist older versions of gdb

2018-04-10 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9962 ) Change subject: pstack_watcher: blacklist older versions of gdb .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] releasing: update gerrit update instructions

2018-04-10 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/9950 ) Change subject: releasing: update gerrit update instructions .. Patch Set 1: Code-Review+2 (1 comment)

[kudu-CR] [tools] ksck improvements [3/n]: master consensus checks

2018-04-10 Thread Will Berkeley (Code Review)
Hello Mike Percy, Attila Bukor, Kudu Jenkins, Andrew Wong, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9883 to look at the new patch set (#7). Change subject: [tools] ksck improvements [3/n]: master consensus checks

[kudu-CR] KUDU-2191 (9/n): HMS Catalog should short-circuit no-op alter and create table ops

2018-04-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9965 ) Change subject: KUDU-2191 (9/n): HMS Catalog should short-circuit no-op alter and create table ops .. Patch Set 2: (8 comments)

[kudu-CR] user: fix a possible integer overflow

2018-04-10 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Jean-Daniel Cryans, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/9975 to review the following change. Change subject: user: fix a possible integer overflow .. user:

[kudu-CR] [docs] Add docs on full data dirs

2018-04-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9967 ) Change subject: [docs] Add docs on full data dirs .. Patch Set 1: (2 comments)

[kudu-CR] [docs] Add docs on full data dirs

2018-04-10 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9967 to look at the new patch set (#2). Change subject: [docs] Add docs on full data dirs ..

[kudu-CR] [docs] Add docs on full data dirs

2018-04-10 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/9967 ) Change subject: [docs] Add docs on full data dirs .. Patch Set 1: (4 comments)

[kudu-CR] [tools] ksck improvements [3/n]: master consensus checks

2018-04-10 Thread Will Berkeley (Code Review)
Will Berkeley has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/9883 ) Change subject: [tools] ksck improvements [3/n]: master consensus checks .. Removed reviewer Kudu Jenkins with the following

[kudu-CR] pstack watcher: blacklist older versions of gdb

2018-04-10 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9962 to look at the new patch set (#3). Change subject: pstack_watcher: blacklist older versions of gdb

[kudu-CR] KUDU-2191 (9/n): HMS Catalog should short-circuit no-op alter and create table calls

2018-04-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9965 ) Change subject: KUDU-2191 (9/n): HMS Catalog should short-circuit no-op alter and create table calls .. Patch Set 3: Code-Review+2 (3

[kudu-CR] [tools] ksck improvements [4/n]: KUDU-2271 and KUDU-2310

2018-04-10 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/9912 ) Change subject: [tools] ksck improvements [4/n]: KUDU-2271 and KUDU-2310 .. Patch Set 5: Code-Review+1 -- To view, visit

[kudu-CR] KUDU-2191 (9/n): HMS Catalog should short-circuit no-op alter and create table calls

2018-04-10 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9965 ) Change subject: KUDU-2191 (9/n): HMS Catalog should short-circuit no-op alter and create table calls .. KUDU-2191 (9/n): HMS

[kudu-CR] user: fix a possible integer overflow

2018-04-10 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9975 ) Change subject: user: fix a possible integer overflow .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] [tools] ksck improvements [3/n]: master consensus checks

2018-04-10 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9883 ) Change subject: [tools] ksck improvements [3/n]: master consensus checks .. [tools] ksck improvements [3/n]: master consensus

[kudu-CR] [tools] ksck improvements [4/n]: KUDU-2271 and KUDU-2310

2018-04-10 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/9912 ) Change subject: [tools] ksck improvements [4/n]: KUDU-2271 and KUDU-2310 .. Patch Set 5: Verified+1 Unrelated flake and incorrect

[kudu-CR] user: fix a possible integer overflow

2018-04-10 Thread Adar Dembo (Code Review)
Hello Tidy Bot, Dan Burkert, Jean-Daniel Cryans, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9975 to look at the new patch set (#2). Change subject: user: fix a possible integer overflow

[kudu-CR] user: fix a possible integer overflow

2018-04-10 Thread Adar Dembo (Code Review)
Hello Tidy Bot, Dan Burkert, Jean-Daniel Cryans, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9975 to look at the new patch set (#3). Change subject: user: fix a possible integer overflow

[kudu-CR] [tools] ksck improvements [3/n]: master consensus checks

2018-04-10 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/9883 ) Change subject: [tools] ksck improvements [3/n]: master consensus checks .. Patch Set 8: Verified+1 (4 comments) IWYU's suggestion is

[kudu-CR] user: fix a possible integer overflow

2018-04-10 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9975 ) Change subject: user: fix a possible integer overflow .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] [tools] ksck improvements [3/n]: master consensus checks

2018-04-10 Thread Will Berkeley (Code Review)
Hello Mike Percy, Attila Bukor, Kudu Jenkins, Andrew Wong, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9883 to look at the new patch set (#8). Change subject: [tools] ksck improvements [3/n]: master consensus checks

[kudu-CR] Add GetFlags endpoint and tool

2018-04-10 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9948 ) Change subject: Add GetFlags endpoint and tool .. Add GetFlags endpoint and tool This adds an rpc endpoint that retrieves

[kudu-CR] [docs] Add docs on full data dirs

2018-04-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9967 ) Change subject: [docs] Add docs on full data dirs .. Patch Set 2: (3 comments)

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-10 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8313 to look at the new patch set (#4). Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener

[kudu-CR] KUDU-2191 (9/n): HMS Catalog should short-circuit no-op alter and create table calls

2018-04-10 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9965 ) Change subject: KUDU-2191 (9/n): HMS Catalog should short-circuit no-op alter and create table calls .. Patch Set 2: (8 comments)

[kudu-CR] KUDU-2191 (9/n): HMS Catalog should short-circuit no-op alter and create table calls

2018-04-10 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9965 to look at the new patch set (#3). Change subject: KUDU-2191 (9/n): HMS Catalog should short-circuit no-op alter and create table calls

[kudu-CR] KUDU-2191 (9/n): HMS Catalog should short-circuit no-op alter and create table calls

2018-04-10 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9965 ) Change subject: KUDU-2191 (9/n): HMS Catalog should short-circuit no-op alter and create table calls .. Patch Set 3: Code-Review+2 -- To

[kudu-CR] [tools] ksck improvements [3/n]: master consensus checks

2018-04-10 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/9883 ) Change subject: [tools] ksck improvements [3/n]: master consensus checks .. Patch Set 8: Code-Review+1 -- To view, visit

[kudu-CR] [tools] ksck improvements [3/n]: master consensus checks

2018-04-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9883 ) Change subject: [tools] ksck improvements [3/n]: master consensus checks .. Patch Set 8: Code-Review+2 (1 comment)

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-10 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8313 ) Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener .. Patch Set 4: (24 comments)

[kudu-CR] [spark] KUDU-2371: Add KuduWriteOptions class and ignoreNull option

2018-04-10 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9834 ) Change subject: [spark] KUDU-2371: Add KuduWriteOptions class and ignoreNull option .. Patch Set 6: Code-Review+1 (2 comments) Overall it

[kudu-CR] [tools] ksck improvements [4/n]: KUDU-2271 and KUDU-2310

2018-04-10 Thread Will Berkeley (Code Review)
Will Berkeley has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/9912 ) Change subject: [tools] ksck improvements [4/n]: KUDU-2271 and KUDU-2310 .. Removed reviewer Kudu Jenkins with the following

[kudu-CR] releasing: update gerrit update instructions

2018-04-10 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change. Change subject: releasing: update gerrit update instructions .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/9950 To unsubscribe,

[kudu-CR] releasing: update gerrit update instructions

2018-04-10 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/9950 ) Change subject: releasing: update gerrit update instructions .. Patch Set 1: Verified+1 -- To view, visit

[kudu-CR] Add GetFlags endpoint and tool

2018-04-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9948 ) Change subject: Add GetFlags endpoint and tool .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/9948 To

[kudu-CR] [spark] KUDU-2371: Add KuduWriteOptions class and ignoreNull option

2018-04-10 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9834 ) Change subject: [spark] KUDU-2371: Add KuduWriteOptions class and ignoreNull option .. Patch Set 6: Nice! I like where this has ended

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8313 ) Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener .. Patch Set 6: (25 comments)

[kudu-CR] [jepsen] update pattern for kudu-master-see-tservers

2018-04-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9978 ) Change subject: [jepsen] update pattern for kudu-master-see-tservers .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2351 Add IP/port for Recv() failure

2018-04-10 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9818 ) Change subject: KUDU-2351 Add IP/port for Recv() failure .. Patch Set 10: > Patch Set 10: > > > Patch Set 10: > > > > > Patch Set 10: >

[kudu-CR] [jepsen] update pattern for kudu-master-see-tservers

2018-04-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9978 ) Change subject: [jepsen] update pattern for kudu-master-see-tservers .. [jepsen] update pattern for kudu-master-see-tservers

[kudu-CR] [spark] KUDU-2371: Add KuduWriteOptions class and ignoreNull option

2018-04-10 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9834 ) Change subject: [spark] KUDU-2371: Add KuduWriteOptions class and ignoreNull option .. Patch Set 6: (5 comments)

[kudu-CR] [jepsen] update pattern for kudu-master-see-tservers

2018-04-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9978 Change subject: [jepsen] update pattern for kudu-master-see-tservers .. [jepsen] update pattern for kudu-master-see-tservers Changelist

[kudu-CR] KUDU-2293 fix cleanup after failed copies

2018-04-10 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9452 ) Change subject: KUDU-2293 fix cleanup after failed copies .. Patch Set 10: (1 comment)

[kudu-CR] pstack watcher: blacklist older versions of gdb

2018-04-10 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9962 ) Change subject: pstack_watcher: blacklist older versions of gdb .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] docs: Add scripts to automate web site update for releases

2018-04-10 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8800 ) Change subject: docs: Add scripts to automate web site update for releases .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-10 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8313 to look at the new patch set (#6). Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener

[kudu-CR] [tools] ksck improvements [4/n]: KUDU-2271 and KUDU-2310

2018-04-10 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9912 ) Change subject: [tools] ksck improvements [4/n]: KUDU-2271 and KUDU-2310 .. [tools] ksck improvements [4/n]: KUDU-2271 and

[kudu-CR] [tools] ksck improvements [4/n]: KUDU-2271 and KUDU-2310

2018-04-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9912 ) Change subject: [tools] ksck improvements [4/n]: KUDU-2271 and KUDU-2310 .. Patch Set 5: Code-Review+2 Will the IWYU errors continue to

[kudu-CR] [docs] Add docs on full data dirs

2018-04-10 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/9967 ) Change subject: [docs] Add docs on full data dirs .. Patch Set 2: (2 comments)

[kudu-CR] [docs] Add docs on full data dirs

2018-04-10 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9967 to look at the new patch set (#3). Change subject: [docs] Add docs on full data dirs ..

[kudu-CR] KUDU-2293 fix cleanup after failed copies

2018-04-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9452 ) Change subject: KUDU-2293 fix cleanup after failed copies .. Patch Set 10: (1 comment)

[kudu-CR] [tools] ksck improvements [4/n]: KUDU-2271 and KUDU-2310

2018-04-10 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/9912 ) Change subject: [tools] ksck improvements [4/n]: KUDU-2271 and KUDU-2310 .. Patch Set 5: > Could you put this on Alexey's radar? Will

[kudu-CR] user: fix a possible integer overflow

2018-04-10 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9975 ) Change subject: user: fix a possible integer overflow .. user: fix a possible integer overflow This was flagged by a run of

[kudu-CR] [tools] ksck improvements [4/n]: KUDU-2271 and KUDU-2310

2018-04-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9912 ) Change subject: [tools] ksck improvements [4/n]: KUDU-2271 and KUDU-2310 .. Patch Set 6: > Patch Set 5: > > > Could you put this on

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-10 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8313 to look at the new patch set (#5). Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener

[kudu-CR] [docs] Add docs on full data dirs

2018-04-10 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9967 Change subject: [docs] Add docs on full data dirs .. [docs] Add docs on full data dirs Change-Id:

[kudu-CR] [docs] Add docs on full data dirs

2018-04-10 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/9967 ) Change subject: [docs] Add docs on full data dirs .. Patch Set 1: Wasn't sure on all the details so please correct me or direct me to

[kudu-CR] Add GetFlags endpoint and tool

2018-04-10 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/9948 ) Change subject: Add GetFlags endpoint and tool .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/9948/3//COMMIT_MSG

[kudu-CR] Add GetFlags endpoint and tool

2018-04-10 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9948 to look at the new patch set (#4). Change subject: Add GetFlags endpoint and tool

[kudu-CR] [KUDU-2214] Voting while tablet copying says "voting while tombstoned"

2018-04-10 Thread Will Berkeley (Code Review)
Will Berkeley has abandoned this change. ( http://gerrit.cloudera.org:8080/8666 ) Change subject: [KUDU-2214] Voting while tablet copying says "voting while tombstoned" .. Abandoned I never finished this, and I think

[kudu-CR] KUDU-2293 fix cleanup after failed copies

2018-04-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9452 ) Change subject: KUDU-2293 fix cleanup after failed copies .. Patch Set 11: (3 comments)

[kudu-CR] KUDU-2293 fix cleanup after failed copies

2018-04-10 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9452 to look at the new patch set (#11). Change subject: KUDU-2293 fix cleanup after failed copies

[kudu-CR] KUDU-2293 fix cleanup after failed copies

2018-04-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9452 ) Change subject: KUDU-2293 fix cleanup after failed copies .. Patch Set 12: (3 comments)

[kudu-CR] KUDU-2293 fix cleanup after failed copies

2018-04-10 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9452 to look at the new patch set (#12). Change subject: KUDU-2293 fix cleanup after failed copies

[kudu-CR] KUDU-2293 fix cleanup after failed copies

2018-04-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9452 ) Change subject: KUDU-2293 fix cleanup after failed copies .. Patch Set 12: IWYU is telling me to remove "kudu/gutil/callback.h" from

[kudu-CR] [docs] Add docs on full data dirs

2018-04-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9967 ) Change subject: [docs] Add docs on full data dirs .. Patch Set 3: (2 comments)

[kudu-CR] KUDU-2293 fix cleanup after failed copies

2018-04-10 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9452 ) Change subject: KUDU-2293 fix cleanup after failed copies .. KUDU-2293 fix cleanup after failed copies Before, when a tablet

[kudu-CR] KUDU-2293 fix cleanup after failed copies

2018-04-10 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9452 ) Change subject: KUDU-2293 fix cleanup after failed copies .. Patch Set 10: (1 comment)

[kudu-CR] KUDU-2293 fix cleanup after failed copies

2018-04-10 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9452 ) Change subject: KUDU-2293 fix cleanup after failed copies .. Patch Set 11: (1 comment)

[kudu-CR] KUDU-2293 fix cleanup after failed copies

2018-04-10 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9452 ) Change subject: KUDU-2293 fix cleanup after failed copies .. Patch Set 12: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2293 fix cleanup after failed copies

2018-04-10 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9452 ) Change subject: KUDU-2293 fix cleanup after failed copies .. Patch Set 11: (2 comments)

[kudu-CR] KUDU-2293 fix cleanup after failed copies

2018-04-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9452 ) Change subject: KUDU-2293 fix cleanup after failed copies .. Patch Set 12: Verified+1 The test failures seem to be unrelated (tsan

[kudu-CR] KUDU-2293 fix cleanup after failed copies

2018-04-10 Thread Andrew Wong (Code Review)
Andrew Wong has removed a vote on this change. Change subject: KUDU-2293 fix cleanup after failed copies .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/9452 To unsubscribe, visit