[kudu-CR] [ttl cache] allow outstanding handles

2019-04-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13112 ) Change subject: [ttl_cache] allow outstanding handles .. Patch Set 1: (1 comment)

[kudu-CR] [authz] new SentryAuthzProvider's caching strategy

2019-04-25 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13069 ) Change subject: [authz] new SentryAuthzProvider's caching strategy .. Patch Set 9: Code-Review+2 (1 comment) LGTM, just one minor nit.

[kudu-CR](branch-1.9.x) [docs] Add maximum tablet size recommendations

2019-04-25 Thread Grant Henke (Code Review)
Grant Henke has abandoned this change. ( http://gerrit.cloudera.org:8080/13106 ) Change subject: [docs] Add maximum tablet size recommendations .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/13106 To

[kudu-CR] [test] Add testOpenScanWithDroppedPartition test

2019-04-25 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13109 ) Change subject: [test] Add testOpenScanWithDroppedPartition test .. Patch Set 2: (4 comments)

[kudu-CR](branch-1.9.x) [test] Add testOpenScanWithDroppedPartition test

2019-04-25 Thread Grant Henke (Code Review)
Grant Henke has abandoned this change. ( http://gerrit.cloudera.org:8080/13116 ) Change subject: [test] Add testOpenScanWithDroppedPartition test .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/13116 To

[kudu-CR](branch-1.9.x) [test] Add testOpenScanWithDroppedPartition test

2019-04-25 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13116 Change subject: [test] Add testOpenScanWithDroppedPartition test .. [test] Add testOpenScanWithDroppedPartition test Adds a test that

[kudu-CR] [test] Add testOpenScanWithDroppedPartition test

2019-04-25 Thread Grant Henke (Code Review)
Hello Mike Percy, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13109 to look at the new patch set (#3). Change subject: [test] Add testOpenScanWithDroppedPartition test

[kudu-CR] [test] Add testOpenScanWithDroppedPartition test

2019-04-25 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13109 ) Change subject: [test] Add testOpenScanWithDroppedPartition test .. Patch Set 3: Verified+1 -- To view, visit

[kudu-CR] [test] Add testOpenScanWithDroppedPartition test

2019-04-25 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change. Change subject: [test] Add testOpenScanWithDroppedPartition test .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13109 To unsubscribe,

[kudu-CR] KUDU-2790: Enhance the backup and restore tests

2019-04-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13120 ) Change subject: KUDU-2790: Enhance the backup and restore tests .. Patch Set 2: (2 comments) I didn't review the new tests super

[kudu-CR] KUDU-2790: Enhance the backup and restore tests

2019-04-25 Thread Grant Henke (Code Review)
Hello Mike Percy, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13120 to look at the new patch set (#3). Change subject: KUDU-2790: Enhance the backup and restore tests

[kudu-CR] KUDU-2790: Enhance the backup and restore tests

2019-04-25 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13120 ) Change subject: KUDU-2790: Enhance the backup and restore tests .. Patch Set 2: (1 comment)

[kudu-CR] [backup] Remove suffix default of “-restore”

2019-04-25 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13121 ) Change subject: [backup] Remove suffix default of “-restore” .. Patch Set 2: (1 comment)

[kudu-CR] [test] Deflake TestKuduScanner.testDiffScan

2019-04-25 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13123 to look at the new patch set (#2). Change subject: [test] Deflake TestKuduScanner.testDiffScan ..

[kudu-CR] [master] add RPC to reset AuthzProvider's cache

2019-04-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13063 ) Change subject: [master] add RPC to reset AuthzProvider's cache .. Patch Set 3: (5 comments)

[kudu-CR] KUDU-2790: Enhance the backup and restore tests

2019-04-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13120 ) Change subject: KUDU-2790: Enhance the backup and restore tests .. Patch Set 3: (1 comment)

[kudu-CR] [test] Deflake TestKuduScanner.testDiffScan

2019-04-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13123 ) Change subject: [test] Deflake TestKuduScanner.testDiffScan .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] [backup] Remove suffix default of “-restore”

2019-04-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13121 ) Change subject: [backup] Remove suffix default of “-restore” .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2790: Enhance the backup and restore tests

2019-04-25 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13120 ) Change subject: KUDU-2790: Enhance the backup and restore tests .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-2790: Enhance the backup and restore tests

2019-04-25 Thread Grant Henke (Code Review)
Hello Mike Percy, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13120 to look at the new patch set (#4). Change subject: KUDU-2790: Enhance the backup and restore tests

[kudu-CR] [test] Deflake TestKuduScanner.testDiffScan

2019-04-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13123 ) Change subject: [test] Deflake TestKuduScanner.testDiffScan .. Patch Set 1: (2 comments)

[kudu-CR] [test] Add testOpenScanWithDroppedPartition test

2019-04-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13109 ) Change subject: [test] Add testOpenScanWithDroppedPartition test .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2790: Enhance the backup and restore tests

2019-04-25 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13120 ) Change subject: KUDU-2790: Enhance the backup and restore tests .. Patch Set 2: (1 comment)

[kudu-CR] hms: remove experimental tag from critical flags

2019-04-25 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13124 Change subject: hms: remove experimental tag from critical flags .. hms: remove experimental tag from critical flags The HMS integration

[kudu-CR] [backup] Remove suffix default of “-restore”

2019-04-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13121 ) Change subject: [backup] Remove suffix default of “-restore” .. Patch Set 2: (2 comments)

[kudu-CR] [backup] Remove suffix default of “-restore”

2019-04-25 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13121 ) Change subject: [backup] Remove suffix default of “-restore” .. Patch Set 2: (3 comments)

[kudu-CR] [backup] Remove suffix default of “-restore”

2019-04-25 Thread Grant Henke (Code Review)
Hello Mike Percy, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13121 to look at the new patch set (#3). Change subject: [backup] Remove suffix default of “-restore”

[kudu-CR] hms: remove experimental tag from critical flags

2019-04-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13124 ) Change subject: hms: remove experimental tag from critical flags .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2790: Enhance the backup and restore tests

2019-04-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13120 ) Change subject: KUDU-2790: Enhance the backup and restore tests .. Patch Set 4: Code-Review+2 -- To view, visit

[kudu-CR] [test] Deflake TestKuduScanner.testDiffScan

2019-04-25 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13123 ) Change subject: [test] Deflake TestKuduScanner.testDiffScan .. [test] Deflake TestKuduScanner.testDiffScan In the case there

[kudu-CR] [master] add RPC to reset AuthzProvider's cache

2019-04-25 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Andrew Wong, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13063 to look at the new patch set (#4). Change subject: [master] add RPC to reset AuthzProvider's cache

[kudu-CR] [master] add RPC to reset AuthzProvider's cache

2019-04-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13063 ) Change subject: [master] add RPC to reset AuthzProvider's cache .. Patch Set 4: Code-Review+1 (2 comments)

[kudu-CR] [master] add RPC to reset AuthzProvider's cache

2019-04-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13063 ) Change subject: [master] add RPC to reset AuthzProvider's cache .. Patch Set 3: (5 comments)

[kudu-CR] [master] add RPC to reset AuthzProvider's cache

2019-04-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13063 ) Change subject: [master] add RPC to reset AuthzProvider's cache .. Patch Set 4: (2 comments)

[kudu-CR] [master] add RPC to reset AuthzProvider's cache

2019-04-25 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Andrew Wong, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13063 to look at the new patch set (#5). Change subject: [master] add RPC to reset AuthzProvider's cache

[kudu-CR] [ttl cache] allow outstanding handles

2019-04-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13112 ) Change subject: [ttl_cache] allow outstanding handles .. Patch Set 1: (3 comments)

[kudu-CR] KUDU-2790: Enhance the backup and restore tests

2019-04-25 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13120 ) Change subject: KUDU-2790: Enhance the backup and restore tests .. KUDU-2790: Enhance the backup and restore tests This patch

[kudu-CR] KUDU-2790: Enhance the backup and restore tests

2019-04-25 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change. Change subject: KUDU-2790: Enhance the backup and restore tests .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13120 To unsubscribe,

[kudu-CR] [master] add RPC to reset AuthzProvider's cache

2019-04-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13063 ) Change subject: [master] add RPC to reset AuthzProvider's cache .. Patch Set 5: Verified+1 Unrelated tests flakies. DEBUG:

[kudu-CR] KUDU-2790: Enhance the backup and restore tests

2019-04-25 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13120 ) Change subject: KUDU-2790: Enhance the backup and restore tests .. Patch Set 4: Verified+1 -- To view, visit

[kudu-CR] hms: remove experimental tag from critical flags

2019-04-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13124 ) Change subject: hms: remove experimental tag from critical flags .. Patch Set 1: (1 comment)

[kudu-CR] [authz] new SentryAuthzProvider's caching strategy

2019-04-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13069 ) Change subject: [authz] new SentryAuthzProvider's caching strategy .. [authz] new SentryAuthzProvider's caching strategy This

[kudu-CR] [authz] new SentryAuthzProvider's caching strategy

2019-04-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13069 ) Change subject: [authz] new SentryAuthzProvider's caching strategy .. Patch Set 9: (1 comment)

[kudu-CR] [ttl cache] allow outstanding handles

2019-04-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13112 ) Change subject: [ttl_cache] allow outstanding handles .. Patch Set 1: (1 comment)

[kudu-CR](branch-1.9.x) [docs] Add maximum tablet size recommendations

2019-04-25 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13115 ) Change subject: [docs] Add maximum tablet size recommendations .. [docs] Add maximum tablet size recommendations Based

[kudu-CR](branch-1.9.x) [docs] Add maximum tablet size recommendations

2019-04-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13115 ) Change subject: [docs] Add maximum tablet size recommendations .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [ttl cache] allow outstanding handles

2019-04-25 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13112 ) Change subject: [ttl_cache] allow outstanding handles .. Patch Set 1: (2 comments)

[kudu-CR] [master] add RPC to reset AuthzProvider's cache

2019-04-25 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Andrew Wong, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13063 to look at the new patch set (#3). Change subject: [master] add RPC to reset AuthzProvider's cache

[kudu-CR] [master] add RPC to reset AuthzProvider's cache

2019-04-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13063 ) Change subject: [master] add RPC to reset AuthzProvider's cache .. Patch Set 2: (13 comments)

[kudu-CR] KUDU-2790: Enhance the backup and restore tests

2019-04-25 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13120 Change subject: KUDU-2790: Enhance the backup and restore tests .. KUDU-2790: Enhance the backup and restore tests This patch enhances

[kudu-CR] [test] Deflake TestKuduScanner.testDiffScan

2019-04-25 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13123 Change subject: [test] Deflake TestKuduScanner.testDiffScan .. [test] Deflake TestKuduScanner.testDiffScan In the case there were 0

[kudu-CR] [backup] Remove suffix default of “-restore”

2019-04-25 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13121 Change subject: [backup] Remove suffix default of “-restore” .. [backup] Remove suffix default of “-restore” Users expect to restore the

[kudu-CR] KUDU-2790: Enhance the backup and restore tests

2019-04-25 Thread Grant Henke (Code Review)
Hello Mike Percy, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13120 to look at the new patch set (#2). Change subject: KUDU-2790: Enhance the backup and restore tests

[kudu-CR] [master] add RPC to reset AuthzProvider's cache

2019-04-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13063 ) Change subject: [master] add RPC to reset AuthzProvider's cache .. Patch Set 5: Code-Review+1 -- To view, visit

[kudu-CR] [backup] Remove suffix default of “-restore”

2019-04-25 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/13121 ) Change subject: [backup] Remove suffix default of “-restore” .. Patch Set 4: Code-Review+2 -- To view, visit

[kudu-CR] [backup] Remove suffix default of “-restore”

2019-04-25 Thread Mike Percy (Code Review)
Mike Percy has removed a vote on this change. Change subject: [backup] Remove suffix default of “-restore” .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13121 To unsubscribe,

[kudu-CR] [backup] Remove suffix default of “-restore”

2019-04-25 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13121 ) Change subject: [backup] Remove suffix default of “-restore” .. [backup] Remove suffix default of “-restore” Users expect to

[kudu-CR] [backup] Remove suffix default of “-restore”

2019-04-25 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/13121 ) Change subject: [backup] Remove suffix default of “-restore” .. Patch Set 4: Verified+1 Unrelated test flake -- To view, visit

[kudu-CR] NOT FOR COMMIT [ttl cache] allow outstanding handles

2019-04-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/13112 ) Change subject: NOT FOR COMMIT [ttl_cache] allow outstanding handles .. Removed reviewer Kudu Jenkins with the following

[kudu-CR] NOT FOR COMMIT [ttl cache] allow outstanding handles

2019-04-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13112 ) Change subject: NOT FOR COMMIT [ttl_cache] allow outstanding handles .. Patch Set 2: Verified+1 Unrelated flakes in Java tests:

[kudu-CR] [master] add RPC to reset AuthzProvider's cache

2019-04-25 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13063 ) Change subject: [master] add RPC to reset AuthzProvider's cache .. Patch Set 5: Code-Review+1 (6 comments) Just few nits.

[kudu-CR] [authz] more tests for SentryPrivilegesFetcher's cache

2019-04-25 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13125 ) Change subject: [authz] more tests for SentryPrivilegesFetcher's cache .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [master] add RPC to reset AuthzProvider's cache

2019-04-25 Thread Alexey Serbin (Code Review)
Hello Andrew Wong, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13063 to look at the new patch set (#6). Change subject: [master] add RPC to reset AuthzProvider's cache

[kudu-CR] [master] add RPC to reset AuthzProvider's cache

2019-04-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13063 ) Change subject: [master] add RPC to reset AuthzProvider's cache .. Patch Set 5: (6 comments)

[kudu-CR] [authz] more tests for SentryPrivilegesFetcher's cache

2019-04-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13125 Change subject: [authz] more tests for SentryPrivilegesFetcher's cache .. [authz] more tests for SentryPrivilegesFetcher's cache This

[kudu-CR] [authz] more tests for SentryPrivilegesFetcher's cache

2019-04-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13125 ) Change subject: [authz] more tests for SentryPrivilegesFetcher's cache .. [authz] more tests for SentryPrivilegesFetcher's