[kudu-CR] Add release notes for tablet history GC

2016-09-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add release notes for tablet history GC .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/3988/6/docs/release_notes.adoc File

[kudu-CR] Add release notes for tablet history GC

2016-09-10 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add release notes for tablet history GC .. Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/3988/5/docs/release_notes.adoc File

[kudu-CR] Add release notes for tablet history GC

2016-09-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add release notes for tablet history GC .. Patch Set 7: Build Started http://104.196.14.100/job/kudu-gerrit/3337/ -- To view, visit

[kudu-CR] Add release notes for tablet history GC

2016-09-10 Thread Mike Percy (Code Review)
Hello Will Berkeley, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3988 to look at the new patch set (#7). Change subject: Add release notes for tablet history GC .. Add

[kudu-CR] tool: port kudu-fs dump, remove kudu-fs list

2016-09-10 Thread Dinesh Bhat (Code Review)
Hello Adar Dembo, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4305 to look at the new patch set (#6). Change subject: tool: port kudu-fs_dump, remove kudu-fs_list

[kudu-CR] KUDU-236 (part 3). Enable tablet history GC by default

2016-09-10 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3983 to look at the new patch set (#6). Change subject: KUDU-236 (part 3). Enable tablet history GC by default ..

[kudu-CR] KUDU-236 (part 2). Create randomized tablet history GC itest

2016-09-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-236 (part 2). Create randomized tablet history GC itest .. Patch Set 13: Build Started http://104.196.14.100/job/kudu-gerrit// -- To view, visit

[kudu-CR] Separate tablet MM ops into separate .cc file

2016-09-10 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4362 to look at the new patch set (#2). Change subject: Separate tablet MM ops into separate .cc file .. Separate

[kudu-CR] Add release notes for tablet history GC

2016-09-10 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add release notes for tablet history GC .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/3988/5/docs/release_notes.adoc File

[kudu-CR] Add release notes for tablet history GC

2016-09-10 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add release notes for tablet history GC .. Patch Set 5: > is there somewhere that this also belongs in the docs? Yeah, I'll dig around for where to include this

[kudu-CR] KUDU-236 (part 3). Enable tablet history GC by default

2016-09-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-236 (part 3). Enable tablet history GC by default .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3983 To unsubscribe, visit

[kudu-CR] KUDU-236 (part 2). Create randomized tablet history GC itest

2016-09-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-236 (part 2). Create randomized tablet history GC itest .. Patch Set 13: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3975 To

[kudu-CR] Add RegexpKuduOperationsProducer class

2016-09-10 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add RegexpKuduOperationsProducer class .. Patch Set 4: Code-Review+2 (7 comments) Cool idea!

[kudu-CR] Add RegexpKuduOperationsProducer class

2016-09-10 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add RegexpKuduOperationsProducer class .. Patch Set 4: -Code-Review Oops, a couple things to tweak before commit I think -- To view, visit

[kudu-CR] tool: port kudu-fs dump, remove kudu-fs list

2016-09-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: tool: port kudu-fs_dump, remove kudu-fs_list .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/4305/5/src/kudu/tools/tool_action_common.h File

[kudu-CR] [python] Implement Scan Token API

2016-09-10 Thread Jordan Birdsell (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4367 to look at the new patch set (#3). Change subject: [python] Implement Scan Token API .. [python] Implement Scan

[kudu-CR] [python] Implement Scan Token API

2016-09-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [python] Implement Scan Token API .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/3341/ -- To view, visit

[kudu-CR] [python] Implement Scan Token API

2016-09-10 Thread Jordan Birdsell (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4367 to look at the new patch set (#2). Change subject: [python] Implement Scan Token API .. [python] Implement Scan

[kudu-CR] python: bump version number to 0.3.0

2016-09-10 Thread Jordan Birdsell (Code Review)
Jordan Birdsell has posted comments on this change. Change subject: python: bump version number to 0.3.0 .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/4364 To unsubscribe, visit

[kudu-CR] python: update cython version check in setup.py

2016-09-10 Thread Jordan Birdsell (Code Review)
Jordan Birdsell has posted comments on this change. Change subject: python: update cython version check in setup.py .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/4368 To unsubscribe, visit

[kudu-CR] [python] Implement Scan Token API

2016-09-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [python] Implement Scan Token API .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3339/ -- To view, visit

[kudu-CR] Add release notes for tablet history GC

2016-09-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Add release notes for tablet history GC .. Add release notes for tablet history GC Change-Id: Ia1ff2322758910041732bed0c74b8a614e6d2cf2 Reviewed-on:

[kudu-CR] Add release notes for tablet history GC

2016-09-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add release notes for tablet history GC .. Patch Set 7: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3988 To unsubscribe, visit

[kudu-CR] [python] Implement Scan Token API

2016-09-10 Thread Jordan Birdsell (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4367 to look at the new patch set (#4). Change subject: [python] Implement Scan Token API .. [python] Implement Scan

[kudu-CR] python: update cython version check in setup.py

2016-09-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: python: update cython version check in setup.py .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3342/ -- To view, visit

[kudu-CR] python: update cython version check in setup.py

2016-09-10 Thread Todd Lipcon (Code Review)
Hello David Ribeiro Alves, Jordan Birdsell, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4368 to review the following change. Change subject: python: update cython version check in setup.py

[kudu-CR] python: bump version number to 0.3.0

2016-09-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: python: bump version number to 0.3.0 .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4364/1/python/setup.py File python/setup.py: PS1, Line 31:

[kudu-CR] [python] Implement Scan Token API

2016-09-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [python] Implement Scan Token API .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/3343/ -- To view, visit

[kudu-CR] Reorganize and fill out release notes for 1.0

2016-09-10 Thread Todd Lipcon (Code Review)
Hello Dan Burkert, Mike Percy, Adar Dembo, Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4369 to review the following change. Change subject: Reorganize and fill out release notes for 1.0

[kudu-CR] Add RegexpKuduOperationsProducer class

2016-09-10 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: Add RegexpKuduOperationsProducer class .. Patch Set 5: Late thought: do we want this in 1.0? If so will make note to Todd to add to his release note rework

[kudu-CR] Add more release notes for new features in 1.0

2016-09-10 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: Add more release notes for new features in 1.0 .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/4370 To unsubscribe, visit

[kudu-CR] Reorganize and fill out release notes for 1.0

2016-09-10 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: Reorganize and fill out release notes for 1.0 .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/4369/1/docs/release_notes.adoc File

[kudu-CR] [python] Implement Scan Token API

2016-09-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [python] Implement Scan Token API .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/3347/ -- To view, visit

[kudu-CR] [python] Implement Scan Token API

2016-09-10 Thread Jordan Birdsell (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4367 to look at the new patch set (#5). Change subject: [python] Implement Scan Token API .. [python] Implement Scan

[kudu-CR] Add RegexpKuduOperationsProducer class

2016-09-10 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: Add RegexpKuduOperationsProducer class .. Patch Set 4: (7 comments) > Cool idea! Actually, I was talking to Jeremy Beard and mentioned that you had suggested

[kudu-CR] Update some comments in Tablet impl and rename a method

2016-09-10 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: Update some comments in Tablet impl and rename a method .. Update some comments in Tablet impl and rename a method Rename DoCompactionOrFlush() to

[kudu-CR] KUDU-236 (part 2). Create randomized tablet history GC itest

2016-09-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-236 (part 2). Create randomized tablet history GC itest .. Patch Set 12: Build Started http://104.196.14.100/job/kudu-gerrit/3332/ -- To view, visit

[kudu-CR] KUDU-236 (part 2). Create randomized tablet history GC itest

2016-09-10 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-236 (part 2). Create randomized tablet history GC itest .. Patch Set 10: (11 comments)

[kudu-CR] KUDU-236 (part 3). Enable tablet history GC by default

2016-09-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-236 (part 3). Enable tablet history GC by default .. Patch Set 6: Build Started http://104.196.14.100/job/kudu-gerrit/3334/ -- To view, visit

[kudu-CR] Add release notes for tablet history GC

2016-09-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add release notes for tablet history GC .. Patch Set 6: Build Started http://104.196.14.100/job/kudu-gerrit/3335/ -- To view, visit

[kudu-CR] master: do not crash on /dump-entities during startup

2016-09-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: master: do not crash on /dump-entities during startup .. master: do not crash on /dump-entities during startup A user reported a master crash when the

[kudu-CR] master: do not crash on /dump-entities during startup

2016-09-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: master: do not crash on /dump-entities during startup .. Patch Set 1: tried the test without the fix locally, looks like it's a good one. -- To view, visit