[kudu-CR] [hms] enable special characters support in mini HMS

2018-09-18 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11467 Change subject: [hms] enable special characters support in mini HMS .. [hms] enable special characters support in mini HMS This commit

[kudu-CR] [hms] relax the notification format restriction

2018-09-18 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11460 ) Change subject: [hms] relax the notification format restriction .. [hms] relax the notification format restriction 'messageFormat'

[kudu-CR] [security] test scenario for KUDU-2580

2018-09-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11448 ) Change subject: [security] test scenario for KUDU-2580 .. Patch Set 3: (10 comments)

[kudu-CR] KUDU-2580 [c++ client] authn token reacquisition fix

2018-09-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11449 ) Change subject: KUDU-2580 [c++ client] authn token reacquisition fix .. Patch Set 1: (6 comments)

[kudu-CR] KUDU-2580 [c++ client] authn token reacquisition fix

2018-09-18 Thread Alexey Serbin (Code Review)
Hello Will Berkeley, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11449 to look at the new patch set (#2). Change subject: KUDU-2580 [c++ client] authn token reacquisition fix

[kudu-CR] [security] test scenario for KUDU-2580

2018-09-18 Thread Alexey Serbin (Code Review)
Hello Will Berkeley, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11448 to look at the new patch set (#3). Change subject: [security] test scenario for KUDU-2580

[kudu-CR](gh-pages) Blogpost describing index skip scan optimization.

2018-09-18 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/11263 ) Change subject: Blogpost describing index skip scan optimization. .. Patch Set 8: (1 comment)

[kudu-CR](gh-pages) Blogpost describing index skip scan optimization.

2018-09-18 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/11263 ) Change subject: Blogpost describing index skip scan optimization. .. Patch Set 8: (1 comment)

[kudu-CR](gh-pages) Blogpost describing index skip scan optimization.

2018-09-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11263 ) Change subject: Blogpost describing index skip scan optimization. .. Patch Set 8: (2 comments)

[kudu-CR](gh-pages) Blogpost describing index skip scan optimization.

2018-09-18 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/11263 ) Change subject: Blogpost describing index skip scan optimization. .. Patch Set 8: (8 comments) Sorry for the delay in reviewing this.

[kudu-CR] [hms] relax the notification format restriction

2018-09-18 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11460 ) Change subject: [hms] relax the notification format restriction .. Patch Set 3: (1 comment)

[kudu-CR] [hms] relax the notification format restriction

2018-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11460 ) Change subject: [hms] relax the notification format restriction .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] [hms] relax the notification format restriction

2018-09-18 Thread Hao Hao (Code Review)
Hello Dan Burkert, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11460 to look at the new patch set (#3). Change subject: [hms] relax the notification format restriction

[kudu-CR] clang tidy gerrit.py: fix output when no changes found in first path

2018-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11457 ) Change subject: clang_tidy_gerrit.py: fix output when no changes found in first path .. clang_tidy_gerrit.py: fix output when no

[kudu-CR] [hms] relax the notification format restriction

2018-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11460 ) Change subject: [hms] relax the notification format restriction .. Patch Set 2: (1 comment)

[kudu-CR] [hms] relax the notification format restriction

2018-09-18 Thread Hao Hao (Code Review)
Hello Dan Burkert, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11460 to look at the new patch set (#2). Change subject: [hms] relax the notification format restriction

[kudu-CR] [hms] relax the notification format restriction

2018-09-18 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11460 ) Change subject: [hms] relax the notification format restriction .. Patch Set 2: (1 comment)

[kudu-CR] [hms] relax the notification format restriction

2018-09-18 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11460 ) Change subject: [hms] relax the notification format restriction .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2463 pt 2: bump MVCC safe time on Raft no-op

2018-09-18 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11427 ) Change subject: KUDU-2463 pt 2: bump MVCC safe time on Raft no-op .. Patch Set 4: Verified+1 The test failure was KUDU-2557,

[kudu-CR] KUDU-2463 pt 2: bump MVCC safe time on Raft no-op

2018-09-18 Thread Andrew Wong (Code Review)
Andrew Wong has removed a vote on this change. Change subject: KUDU-2463 pt 2: bump MVCC safe time on Raft no-op .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/11427 To

[kudu-CR] [hms] relax the notification format restriction

2018-09-18 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11460 ) Change subject: [hms] relax the notification format restriction .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2463 pt 3: don't scan if MVCC hasn't moved

2018-09-18 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11428 ) Change subject: KUDU-2463 pt 3: don't scan if MVCC hasn't moved .. Patch Set 2: (7 comments)

[kudu-CR] KUDU-2463 pt 2: bump MVCC safe time on Raft no-op

2018-09-18 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/11427 to look at the new patch set (#4). Change subject: KUDU-2463 pt 2: bump MVCC safe time on Raft no-op

[kudu-CR] KUDU-2463 pt 3: don't scan if MVCC hasn't moved

2018-09-18 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/11428 to look at the new patch set (#3). Change subject: KUDU-2463 pt 3: don't scan if MVCC hasn't moved

[kudu-CR] [hms] relax the notification format restriction

2018-09-18 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11460 ) Change subject: [hms] relax the notification format restriction .. Patch Set 1: (1 comment)

[kudu-CR] clang tidy gerrit.py: fix output when no changes found in first path

2018-09-18 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11457 ) Change subject: clang_tidy_gerrit.py: fix output when no changes found in first path .. Patch Set 2: Code-Review+2 (1 comment) >

[kudu-CR] clang tidy gerrit.py: fix output when no changes found in first path

2018-09-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11457 ) Change subject: clang_tidy_gerrit.py: fix output when no changes found in first path .. Patch Set 2: Code-Review+2 -- To view,

[kudu-CR] [tests] test scenario for KUDU-2580

2018-09-18 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11448 ) Change subject: [tests] test scenario for KUDU-2580 .. Patch Set 2: (5 comments)

[kudu-CR] [hms] relax the notification format restriction

2018-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11460 ) Change subject: [hms] relax the notification format restriction .. Patch Set 1: (2 comments)

[kudu-CR] clang tidy gerrit.py: fix output when no changes found in first path

2018-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11457 ) Change subject: clang_tidy_gerrit.py: fix output when no changes found in first path .. Patch Set 2: (1 comment)

[kudu-CR] clang tidy gerrit.py: fix output when no changes found in first path

2018-09-18 Thread Adar Dembo (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11457 to look at the new patch set (#2). Change subject: clang_tidy_gerrit.py: fix output when no changes found in first path

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10406 ) Change subject: KUDU-2437 Split a tablet into primary key ranges by size .. KUDU-2437 Split a tablet into primary key ranges by

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10406 ) Change subject: KUDU-2437 Split a tablet into primary key ranges by size .. Patch Set 23: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2580 [c++ client] authn token reacquisition fix

2018-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11449 ) Change subject: KUDU-2580 [c++ client] authn token reacquisition fix .. Patch Set 1: (1 comment)

[kudu-CR] [tests] test scenario for KUDU-2580

2018-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11448 ) Change subject: [tests] test scenario for KUDU-2580 .. Patch Set 2: (5 comments)

[kudu-CR] KUDU-2580 [c++ client] authn token reacquisition fix

2018-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11449 ) Change subject: KUDU-2580 [c++ client] authn token reacquisition fix .. Patch Set 1: (4 comments) If you want to backport it to older

[kudu-CR] KUDU-2463 pt 1: adjust MVCC when replaying no-ops

2018-09-18 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11142 ) Change subject: KUDU-2463 pt 1: adjust MVCC when replaying no-ops .. Patch Set 7: > Patch Set 7: > > (32 comments) Here's a dist test

[kudu-CR] KUDU-2463 pt 1: adjust MVCC when replaying no-ops

2018-09-18 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11142 ) Change subject: KUDU-2463 pt 1: adjust MVCC when replaying no-ops .. Patch Set 7: (32 comments)

[kudu-CR] KUDU-2463 pt 1: adjust MVCC when replaying no-ops

2018-09-18 Thread Andrew Wong (Code Review)
Hello Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11142 to look at the new patch set (#7). Change subject: KUDU-2463 pt 1: adjust MVCC when replaying no-ops ..

[kudu-CR] KUDU-2463 pt 2: bump MVCC safe time on Raft no-op

2018-09-18 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/11427 to look at the new patch set (#3). Change subject: KUDU-2463 pt 2: bump MVCC safe time on Raft no-op