[kudu-CR] [gutil] fix assignment operator signature in DISALLOW COPY AND ASSIGN

2020-07-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16158 ) Change subject: [gutil] fix assignment operator signature in DISALLOW_COPY_AND_ASSIGN .. [gutil] fix assignment operator

[kudu-CR] KUDU-3090 Add ownership privileges

2020-07-09 Thread Attila Bukor (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16072 to look at the new patch set (#18). Change subject: KUDU-3090 Add ownership privileges

[kudu-CR] KUDU-3090: Native owner metadata in Kudu

2020-07-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/15841 ) Change subject: KUDU-3090: Native owner metadata in Kudu .. Patch Set 13: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-3090: Native owner metadata in Kudu

2020-07-09 Thread Attila Bukor (Code Review)
Attila Bukor has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15841 ) Change subject: KUDU-3090: Native owner metadata in Kudu .. KUDU-3090: Native owner metadata in Kudu Apache Sentry and Apache

[kudu-CR] Optimize GetTableLocations for better arena use

2020-07-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/16137 ) Change subject: Optimize GetTableLocations for better arena use .. Patch Set 7: Would it make sense to tag this commit with KUDU-636?

[kudu-CR] [master] set RPC queue length to 100 by default

2020-07-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/16154 ) Change subject: [master] set RPC queue length to 100 by default .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] Add refreshing Ranger policies

2020-07-09 Thread Attila Bukor (Code Review)
Hello Andrew Wong, Greg Solovyev, Grant Henke, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/16160 to review the following change. Change subject: Add refreshing Ranger policies .. Add

[kudu-CR] Use protobuf arenas for CommitMsgs

2020-07-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/16147 ) Change subject: Use protobuf arenas for CommitMsgs .. Patch Set 2: Would it make sense to tag this commit with KUDU-636? -- To view,

[kudu-CR] [master] set RPC queue length to 100 by default

2020-07-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16154 ) Change subject: [master] set RPC queue length to 100 by default .. [master] set RPC queue length to 100 by default There is

[kudu-CR] Add refreshing Ranger policies

2020-07-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/16160 ) Change subject: Add refreshing Ranger policies .. Patch Set 1: (8 comments) http://gerrit.cloudera.org:8080/#/c/16160/1//COMMIT_MSG

[kudu-CR] Add refreshing Ranger policies

2020-07-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/16160 ) Change subject: Add refreshing Ranger policies .. Patch Set 1: (4 comments)

[kudu-CR] [master] set RPC queue length to 100 by default

2020-07-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16154 ) Change subject: [master] set RPC queue length to 100 by default .. Patch Set 1: (1 comment)

[kudu-CR] Add refreshing Ranger policies

2020-07-09 Thread Attila Bukor (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Grant Henke, Greg Solovyev, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16160 to look at the new patch set (#2). Change subject: Add refreshing Ranger policies

[kudu-CR] Add refreshing Ranger policies

2020-07-09 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/16160 ) Change subject: Add refreshing Ranger policies .. Patch Set 1: (7 comments) http://gerrit.cloudera.org:8080/#/c/16160/1//COMMIT_MSG

[kudu-CR] KUDU-636. Use protobuf arenas for CommitMsgs

2020-07-09 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/16147 ) Change subject: KUDU-636. Use protobuf arenas for CommitMsgs .. Patch Set 3: (2 comments) think I fixed the issues but we'll see what

[kudu-CR] KUDU-636. Use protobuf arenas for CommitMsgs

2020-07-09 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/16147 ) Change subject: KUDU-636. Use protobuf arenas for CommitMsgs .. Patch Set 3: also worth noting I rebased this away from the

[kudu-CR] [tests] add same tablet concurrent writes test

2020-07-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16060 ) Change subject: [tests] add same_tablet_concurrent_writes test .. Patch Set 3: (1 comment)

[kudu-CR] [tests] add same tablet concurrent writes test

2020-07-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16060 ) Change subject: [tests] add same_tablet_concurrent_writes test .. Patch Set 4: > Uploaded patch set 4. Whoops, it seems I pushed

[kudu-CR] Add refreshing Ranger policies

2020-07-09 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/16160 ) Change subject: Add refreshing Ranger policies .. Patch Set 3: (5 comments)

[kudu-CR] Add refreshing Ranger policies

2020-07-09 Thread Attila Bukor (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Grant Henke, Greg Solovyev, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16160 to look at the new patch set (#3). Change subject: Add refreshing Ranger policies

[kudu-CR] [tests] add same tablet concurrent writes test

2020-07-09 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16060 to look at the new patch set (#5). Change subject: [tests] add same_tablet_concurrent_writes test

[kudu-CR] KUDU-636. Use protobuf arenas for CommitMsgs

2020-07-09 Thread Todd Lipcon (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16147 to look at the new patch set (#3). Change subject: KUDU-636. Use protobuf arenas for CommitMsgs

[kudu-CR] [tests] add same tablet concurrent writes test

2020-07-09 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16060 to look at the new patch set (#4). Change subject: [tests] add same_tablet_concurrent_writes test

[kudu-CR] Add refreshing Ranger policies

2020-07-09 Thread Attila Bukor (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Grant Henke, Greg Solovyev, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16160 to look at the new patch set (#4). Change subject: Add refreshing Ranger policies

[kudu-CR] KUDU-636. Use protobuf arenas for CommitMsgs

2020-07-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16147 ) Change subject: KUDU-636. Use protobuf arenas for CommitMsgs .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-636. Use protobuf arenas for CommitMsgs

2020-07-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16147 ) Change subject: KUDU-636. Use protobuf arenas for CommitMsgs .. KUDU-636. Use protobuf arenas for CommitMsgs This commit

[kudu-CR](gh-pages) [blog] Cross post from Boris Tyukin

2020-07-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16117 ) Change subject: [blog] Cross post from Boris Tyukin .. Patch Set 3: Code-Review+1 looks good to me; maybe just fix typo

[kudu-CR] KUDU-636. Use Arena for EncodedKeys

2020-07-09 Thread Todd Lipcon (Code Review)
Hello Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/16162 to review the following change. Change subject: KUDU-636. Use Arena for EncodedKeys .. KUDU-636. Use Arena for

[kudu-CR] KUDU-2612 p2: introduce transaction status management

2020-07-09 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16044 ) Change subject: KUDU-2612 p2: introduce transaction status management .. KUDU-2612 p2: introduce transaction status management

[kudu-CR] [docker] KUDU-2857: Rewrite docker build script in python

2020-07-09 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16161 to look at the new patch set (#2). Change subject: [docker] KUDU-2857: Rewrite docker build script in python ..

[kudu-CR] [docker] KUDU-2857: Rewrite docker build script in python

2020-07-09 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16161 Change subject: [docker] KUDU-2857: Rewrite docker build script in python .. [docker] KUDU-2857: Rewrite docker build script in python

[kudu-CR] KUDU-2612 p2: introduce transaction status management

2020-07-09 Thread Andrew Wong (Code Review)
Andrew Wong has removed a vote on this change. Change subject: KUDU-2612 p2: introduce transaction status management .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/16044 To

[kudu-CR] [docker] KUDU-2857: Rewrite docker build script in python

2020-07-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/16161 ) Change subject: [docker] KUDU-2857: Rewrite docker build script in python .. Patch Set 1: I am still running some builds to test this,

[kudu-CR] KUDU-2612 p2: introduce transaction status management

2020-07-09 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16044 ) Change subject: KUDU-2612 p2: introduce transaction status management .. Patch Set 11: Verified+1 timeout of

[kudu-CR] [master] cache for table locations

2020-07-09 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15971 ) Change subject: [master] cache for table locations .. Patch Set 11: Code-Review+2 (1 comment)

[kudu-CR] [maintenance] use workload statistics to scale perf improvement

2020-07-09 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15995 ) Change subject: [maintenance] use workload statistics to scale perf_improvement .. Patch Set 6: (7 comments)

[kudu-CR] Optimize GetTableLocations for better arena use

2020-07-09 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16137 ) Change subject: Optimize GetTableLocations for better arena use .. Patch Set 7: Code-Review+1 (1 comment)

[kudu-CR] [gutil] fix assignment operator signature in DISALLOW COPY AND ASSIGN

2020-07-09 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/16158 ) Change subject: [gutil] fix assignment operator signature in DISALLOW_COPY_AND_ASSIGN .. Patch Set 1: Code-Review+2 -- To view,

[kudu-CR] [master] set RPC queue length to 100 by default

2020-07-09 Thread Bankim Bhavsar (Code Review)
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/16154 ) Change subject: [master] set RPC queue length to 100 by default .. Patch Set 1: Code-Review+1 (1 comment)

[kudu-CR] [master] cache for table locations

2020-07-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15971 ) Change subject: [master] cache for table locations .. Patch Set 10: (1 comment)

[kudu-CR] [master] cache for table locations

2020-07-09 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15971 ) Change subject: [master] cache for table locations .. Patch Set 11: (1 comment)

[kudu-CR] [tablet] OpState::op state lock --> WriteOpState::op state lock

2020-07-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16165 Change subject: [tablet] OpState::op_state_lock_ --> WriteOpState::op_state_lock_ .. [tablet] OpState::op_state_lock_ -->

[kudu-CR] [master] cache for table locations

2020-07-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15971 ) Change subject: [master] cache for table locations .. Patch Set 10: (1 comment)

[kudu-CR] [master] cache for table locations

2020-07-09 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Andrew Wong, Grant Henke, Greg Solovyev, Bankim Bhavsar, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/15971 to look at the new patch set (#12). Change subject: [master] cache for table locations

[kudu-CR] KUDU-636. Use Arena for EncodedKeys

2020-07-09 Thread Todd Lipcon (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/16162 to look at the new patch set (#2). Change subject: KUDU-636. Use Arena for EncodedKeys ..