[kudu-CR] tool: rewrite parser logic

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: tool: rewrite parser logic .. tool: rewrite parser logic While leaf and non-leaf actions share some common properties, there is much they don't share.

[kudu-CR] Add time/watermark based garbage collection to ResultTracker

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add time/watermark based garbage collection to ResultTracker .. Patch Set 17: (1 comment)

[kudu-CR] tool: rewrite parser logic

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: tool: rewrite parser logic .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3996 To unsubscribe, visit

[kudu-CR] Add time/watermark based garbage collection to ResultTracker

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add time/watermark based garbage collection to ResultTracker .. Patch Set 18: Code-Review+2 (1 comment)

[kudu-CR] tool: rewrite parser logic

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: tool: rewrite parser logic .. Patch Set 3: > looks good but the patch it's based on seems to have exploded. Yeah, I rebased to switch the order around. And I

[kudu-CR] tool: rewrite parser logic

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: tool: rewrite parser logic .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3996 To unsubscribe, visit

[kudu-CR] integration tests: end-to-end master permanent failure test

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: integration_tests: end-to-end master permanent failure test .. integration_tests: end-to-end master permanent failure test This commit defines a workflow

[kudu-CR] integration tests: end-to-end master permanent failure test

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: integration_tests: end-to-end master permanent failure test .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3969 To unsubscribe,

[kudu-CR] Add time/watermark based garbage collection to ResultTracker

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add time/watermark based garbage collection to ResultTracker .. Patch Set 18: kicked off dist-test runs with 1000 of each of the integration tests here:

[kudu-CR] Add time/watermark based garbage collection to ResultTracker

2016-08-15 Thread Todd Lipcon (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3628 to look at the new patch set (#18). Change subject: Add time/watermark based garbage collection to ResultTracker

[kudu-CR] Add time/watermark based garbage collection to ResultTracker

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add time/watermark based garbage collection to ResultTracker .. Patch Set 17: (6 comments)

[kudu-CR] Add time/watermark based garbage collection to ResultTracker

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add time/watermark based garbage collection to ResultTracker .. Patch Set 17: Yea, gerrit's poor at dealing with renames. git show -M does a better job. For your

[kudu-CR] KUDU-456 Implement AUTO FLUSH BACKGROUND flush mode

2016-08-15 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3952 to look at the new patch set (#8). Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. KUDU-456

[kudu-CR] KUDU-456 Implement AUTO FLUSH BACKGROUND flush mode

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. Patch Set 8: Build Started http://104.196.14.100/job/kudu-gerrit/2940/ -- To view, visit

[kudu-CR] [util/monotime] added handy operators for MonoTime

2016-08-15 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/3999 Change subject: [util/monotime] added handy operators for MonoTime .. [util/monotime] added handy operators for MonoTime Change-Id:

[kudu-CR] KUDU-236 (part 1). Implement tablet history GC

2016-08-15 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3076 to look at the new patch set (#16). Change subject: KUDU-236 (part 1). Implement tablet history GC .. KUDU-236

[kudu-CR] KUDU-236 (part 1). Implement tablet history GC

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-236 (part 1). Implement tablet history GC .. Patch Set 16: Build Started http://104.196.14.100/job/kudu-gerrit/2938/ -- To view, visit

[kudu-CR] Support downgrade to version that has LocalConsensus

2016-08-15 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Support downgrade to version that has LocalConsensus .. Patch Set 2: > Can you please give this a manual test as part of voting on 0.10 RC? Sure, no problem. --

[kudu-CR] tool: rewrite parser logic

2016-08-15 Thread Adar Dembo (Code Review)
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3996 to look at the new patch set (#2). Change subject: tool: rewrite parser logic .. tool: rewrite parser

[kudu-CR] catalog manager: avoid more races between Init() and GetTabletPeer()

2016-08-15 Thread Adar Dembo (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3997 to review the following change. Change subject: catalog_manager: avoid more races between Init() and GetTabletPeer()

[kudu-CR] tool: rewrite parser logic

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: tool: rewrite parser logic .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3996/1/src/kudu/tools/tool_action.h File src/kudu/tools/tool_action.h:

[kudu-CR] catalog manager: avoid more races between Init() and GetTabletPeer()

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: catalog_manager: avoid more races between Init() and GetTabletPeer() .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2937/ -- To view,

[kudu-CR] fs: allow format with user-specified uuid

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: fs: allow format with user-specified uuid .. fs: allow format with user-specified uuid The CLI tool is going to use this for the "handling permanent

[kudu-CR] fs: allow format with user-specified uuid

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: fs: allow format with user-specified uuid .. Patch Set 5: Verified+1 The test failure was unrelated, and I've got a patch ready to go for it. -- To view, visit

[kudu-CR] KUDU-236 (part 1). Implement tablet history GC

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-236 (part 1). Implement tablet history GC .. Patch Set 15: Build Started http://104.196.14.100/job/kudu-gerrit/2934/ -- To view, visit

[kudu-CR] KUDU-236 (part 1). Implement tablet history GC

2016-08-15 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-236 (part 1). Implement tablet history GC .. Patch Set 14: (1 comment) http://gerrit.cloudera.org:8080/#/c/3076/14/src/kudu/tablet/compaction.cc File

[kudu-CR] tool: rewrite parser logic

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: tool: rewrite parser logic .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3996/1/src/kudu/tools/tool_action.h File

[kudu-CR] integration tests: end-to-end master permanent failure test

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: integration_tests: end-to-end master permanent failure test .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3969 To unsubscribe,

[kudu-CR] fs: allow format with user-specified uuid

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: fs: allow format with user-specified uuid .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/3968/4/src/kudu/util/oid_generator-test.cc File

[kudu-CR] integration tests: end-to-end master permanent failure test

2016-08-15 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3969 to look at the new patch set (#3). Change subject: integration_tests: end-to-end master permanent failure test ..

[kudu-CR] KUDU-1474: single to multi-master deployment migration

2016-08-15 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3880 to look at the new patch set (#7). Change subject: KUDU-1474: single to multi-master deployment migration ..

[kudu-CR] tool: rewrite parser logic

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has uploaded a new change for review. http://gerrit.cloudera.org:8080/3996 Change subject: tool: rewrite parser logic .. tool: rewrite parser logic While leaf and non-leaf actions share some common properties,

[kudu-CR] fs: allow format with user-specified uuid

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: fs: allow format with user-specified uuid .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/2930/ -- To view, visit

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3961 To unsubscribe, visit

[kudu-CR] integration tests: end-to-end master permanent failure test

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: integration_tests: end-to-end master permanent failure test .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/3969/2/src/kudu/tools/tool_action.cc

[kudu-CR] Add time/watermark based garbage collection to ResultTracker

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add time/watermark based garbage collection to ResultTracker .. Patch Set 17: Build Started http://104.196.14.100/job/kudu-gerrit/2929/ -- To view, visit

[kudu-CR] Add time/watermark based garbage collection to ResultTracker

2016-08-15 Thread Todd Lipcon (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3628 to look at the new patch set (#17). Change subject: Add time/watermark based garbage collection to ResultTracker

[kudu-CR] fs: allow format with user-specified uuid

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: fs: allow format with user-specified uuid .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3968 To unsubscribe, visit

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/2928/ -- To view, visit

[kudu-CR] Add time/watermark based garbage collection to ResultTracker

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add time/watermark based garbage collection to ResultTracker .. Patch Set 15: (8 comments)

[kudu-CR] fs: allow format with user-specified uuid

2016-08-15 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3968 to look at the new patch set (#4). Change subject: fs: allow format with user-specified uuid .. fs: allow format

[kudu-CR] fs: allow format with user-specified uuid

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: fs: allow format with user-specified uuid .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3968/2/src/kudu/fs/fs_manager.cc File

[kudu-CR] fs: allow format with user-specified uuid

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: fs: allow format with user-specified uuid .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2926/ -- To view, visit

[kudu-CR] fs: allow format with user-specified uuid

2016-08-15 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3968 to look at the new patch set (#3). Change subject: fs: allow format with user-specified uuid .. fs: allow format

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3961 To unsubscribe, visit

[kudu-CR] Another pass on 0.10.0 release notes

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Another pass on 0.10.0 release notes .. Patch Set 2: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/3979/2/docs/release_notes.adoc File

[kudu-CR] KUDU-526: use on-disk cmeta when loading existing master state

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: KUDU-526: use on-disk cmeta when loading existing master state .. KUDU-526: use on-disk cmeta when loading existing master state The cmeta is populated

[kudu-CR] KUDU-456 Implement AUTO FLUSH BACKGROUND flush mode

2016-08-15 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. Patch Set 6: (7 comments) Thank you for the review! I posted a new version (patchset 7) because the

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2924/ -- To view, visit

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-15 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3961 to look at the new patch set (#4). Change subject: Start a background thread to run ResultTracker GC .. Start a

[kudu-CR] Add time/watermark based garbage collection to ResultTracker

2016-08-15 Thread Todd Lipcon (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3628 to look at the new patch set (#16). Change subject: Add time/watermark based garbage collection to ResultTracker

[kudu-CR] KUDU-1517 Implement doc feedback from Sue M

2016-08-15 Thread Misty Stanley-Jones (Code Review)
Misty Stanley-Jones has posted comments on this change. Change subject: KUDU-1517 Implement doc feedback from Sue M .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3638/1/docs/index.adoc File docs/index.adoc:

[kudu-CR] KUDU-1517 Implement doc feedback from Sue M

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1517 Implement doc feedback from Sue M .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2923/ -- To view, visit

[kudu-CR] Add time/watermark based garbage collection to ResultTracker

2016-08-15 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Add time/watermark based garbage collection to ResultTracker .. Patch Set 15: Code-Review+1 (7 comments)

[kudu-CR] KUDU-1517 Implement doc feedback from Sue M

2016-08-15 Thread Anonymous Coward (Code Review)
Anonymous Coward #149 has posted comments on this change. Change subject: KUDU-1517 Implement doc feedback from Sue M .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3638/1/docs/index.adoc File

[kudu-CR] KUDU-1517 Implement doc feedback from Sue M

2016-08-15 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: KUDU-1517 Implement doc feedback from Sue M .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3638/1/docs/index.adoc File docs/index.adoc: PS1,

[kudu-CR] KUDU-1517 Implement doc feedback from Sue M

2016-08-15 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1517 Implement doc feedback from Sue M .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3638/1/docs/index.adoc File docs/index.adoc: PS1,

[kudu-CR] KUDU-236 (part 1). Implement tablet history GC

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-236 (part 1). Implement tablet history GC .. Patch Set 14: (1 comment) http://gerrit.cloudera.org:8080/#/c/3076/14/src/kudu/tablet/compaction.cc File

[kudu-CR] KUDU-1517 Implement doc feedback from Sue M

2016-08-15 Thread Misty Stanley-Jones (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3638 to look at the new patch set (#2). Change subject: KUDU-1517 Implement doc feedback from Sue M .. KUDU-1517

[kudu-CR] KUDU-1517 Implement doc feedback from Sue M

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1517 Implement doc feedback from Sue M .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2921/ -- To view, visit

[kudu-CR] KUDU-1517 Implement doc feedback from Sue M

2016-08-15 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: KUDU-1517 Implement doc feedback from Sue M .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3638/1/docs/index.adoc File docs/index.adoc: PS1,

[kudu-CR] Another pass on 0.10.0 release notes

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Another pass on 0.10.0 release notes .. Patch Set 1: (2 comments) Did a pass on the new stuff. Looks good overall.

[kudu-CR] Another pass on 0.10.0 release notes

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Another pass on 0.10.0 release notes .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3979/1/docs/release_notes.adoc File

[kudu-CR] KUDU-456 Implement AUTO FLUSH BACKGROUND flush mode

2016-08-15 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3952 to look at the new patch set (#7). Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. KUDU-456

[kudu-CR] KUDU-456 Implement AUTO FLUSH BACKGROUND flush mode

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. Patch Set 7: Build Started http://104.196.14.100/job/kudu-gerrit/2920/ -- To view, visit

[kudu-CR] Another pass on 0.10.0 release notes

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Another pass on 0.10.0 release notes .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3979/1/docs/release_notes.adoc File docs/release_notes.adoc:

[kudu-CR] KUDU-1517 Implement doc feedback from Sue M

2016-08-15 Thread Misty Stanley-Jones (Code Review)
Misty Stanley-Jones has posted comments on this change. Change subject: KUDU-1517 Implement doc feedback from Sue M .. Patch Set 1: (7 comments) http://gerrit.cloudera.org:8080/#/c/3638/1/docs/index.adoc File docs/index.adoc:

[kudu-CR] Improvements and corrections to Impala CREATE TABLE examples

2016-08-15 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Improvements and corrections to Impala CREATE TABLE examples .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3376 To unsubscribe,

[kudu-CR] Improvements and corrections to Impala CREATE TABLE examples

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Improvements and corrections to Impala CREATE TABLE examples .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2919/ -- To view, visit

[kudu-CR] Improvements and corrections to Impala CREATE TABLE examples

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Improvements and corrections to Impala CREATE TABLE examples .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2918/ -- To view, visit

[kudu-CR] Improvements and corrections to Impala CREATE TABLE examples

2016-08-15 Thread Misty Stanley-Jones (Code Review)
Misty Stanley-Jones has posted comments on this change. Change subject: Improvements and corrections to Impala CREATE TABLE examples .. Patch Set 2: (1 comment)

[kudu-CR] Improvements and corrections to Impala CREATE TABLE examples

2016-08-15 Thread Misty Stanley-Jones (Code Review)
Hello Casey Ching, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3376 to look at the new patch set (#3). Change subject: Improvements and corrections to Impala CREATE TABLE examples

[kudu-CR] Add docs for non-covering range partitioning

2016-08-15 Thread Misty Stanley-Jones (Code Review)
Misty Stanley-Jones has posted comments on this change. Change subject: Add docs for non-covering range partitioning .. Patch Set 5: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3796 To unsubscribe, visit

[kudu-CR] Add docs for non-covering range partitioning

2016-08-15 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add docs for non-covering range partitioning .. Patch Set 5: Code-Review+2 (2 comments) http://gerrit.cloudera.org:8080/#/c/3796/3/docs/schema_design.adoc File

[kudu-CR] Add docs for non-covering range partitioning

2016-08-15 Thread Misty Stanley-Jones (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3796 to look at the new patch set (#5). Change subject: Add docs for non-covering range partitioning .. Add docs for

[kudu-CR] Add docs for non-covering range partitioning

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add docs for non-covering range partitioning .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/2917/ -- To view, visit

[kudu-CR] integration tests: end-to-end master permanent failure test

2016-08-15 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: integration_tests: end-to-end master permanent failure test .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-456 Implement AUTO FLUSH BACKGROUND flush mode

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. Patch Set 6: Build Started http://104.196.14.100/job/kudu-gerrit/2916/ -- To view, visit

[kudu-CR] KUDU-456 Implement AUTO FLUSH BACKGROUND flush mode

2016-08-15 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3952 to look at the new patch set (#6). Change subject: KUDU-456 Implement AUTO_FLUSH_BACKGROUND flush mode .. KUDU-456

[kudu-CR] Predicate evaluation pushdown

2016-08-15 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded a new change for review. http://gerrit.cloudera.org:8080/3990 Change subject: Predicate evaluation pushdown .. Predicate evaluation pushdown The premise of this patch is to avoid the excessive use of

[kudu-CR] Predicate evaluation pushdown

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Predicate evaluation pushdown .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2915/ -- To view, visit http://gerrit.cloudera.org:8080/3990

[kudu-CR] fs: allow format with arbitrary uuid

2016-08-15 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: fs: allow format with arbitrary uuid .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3968/2/src/kudu/fs/fs_manager.cc File

[kudu-CR] KUDU-236 (part 1). Implement tablet history GC

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-236 (part 1). Implement tablet history GC .. Patch Set 14: > > Could you also add a GCing thread to mt-tablet-test? It's a good > > way to test interactions

[kudu-CR] Memory tracking for result tracker

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Memory tracking for result tracker .. Patch Set 21: yep, your analysis matches my understanding. -- To view, visit http://gerrit.cloudera.org:8080/3627 To

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/3961/3/src/kudu/rpc/result_tracker.cc File

[kudu-CR] Memory tracking for result tracker

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Memory tracking for result tracker .. Patch Set 21: Code-Review+2 (2 comments) http://gerrit.cloudera.org:8080/#/c/3627/19/src/kudu/rpc/result_tracker.cc File

[kudu-CR] Support downgrade to version that has LocalConsensus

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Support downgrade to version that has LocalConsensus .. Patch Set 1: Code-Review+2 Can you please give this a manual test as part of voting on 0.10 RC? -- To

[kudu-CR] Support downgrade to version that has LocalConsensus

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Support downgrade to version that has LocalConsensus .. Support downgrade to version that has LocalConsensus Continue writing the "local=false" field

[kudu-CR] Support downgrade to version that has LocalConsensus

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Support downgrade to version that has LocalConsensus .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3985 To unsubscribe, visit

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/3961/3/src/kudu/rpc/result_tracker.cc File

[kudu-CR] MergeIterator: don't stop iterating on an empty block

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: MergeIterator: don't stop iterating on an empty block .. MergeIterator: don't stop iterating on an empty block The MergeIterator implementation was

[kudu-CR] Start a background thread to run ResultTracker GC

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Start a background thread to run ResultTracker GC .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/3961/3/src/kudu/rpc/result_tracker.cc File

[kudu-CR] Add docs for non-covering range partitioning

2016-08-15 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3796 to look at the new patch set (#4). Change subject: Add docs for non-covering range partitioning .. Add docs for

[kudu-CR] KUDU-1522 Fix AlterTable Timestamp col with default value

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has abandoned this change. Change subject: KUDU-1522 Fix AlterTable Timestamp col with default value .. Abandoned Yea, looks like the new commit had a different 'Change-Id' line than the original. That causes it

[kudu-CR] KUDU-1522 Fix AlterTable Timestamp col with default value

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1522 Fix AlterTable Timestamp col with default value .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3980 To unsubscribe,

[kudu-CR] KUDU-1522 Fix AlterTable Timestamp col with default value

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1522 Fix AlterTable Timestamp col with default value .. KUDU-1522 Fix AlterTable Timestamp col with default value Change-Id:

[kudu-CR] MergeIterator: don't stop iterating on an empty block

2016-08-15 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: MergeIterator: don't stop iterating on an empty block .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3976 To unsubscribe, visit

[kudu-CR] MergeIterator: don't stop iterating on an empty block

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: MergeIterator: don't stop iterating on an empty block .. Patch Set 2: (1 comment)

[kudu-CR] MergeIterator: don't stop iterating on an empty block

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: MergeIterator: don't stop iterating on an empty block .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2913/ -- To view, visit

[kudu-CR] MergeIterator: don't stop iterating on an empty block

2016-08-15 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3976 to look at the new patch set (#4). Change subject: MergeIterator: don't stop iterating on an empty block ..

  1   2   >