[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 http://gerrit.cloudera.org:8080/43

[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 Tok

[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 patc

[kudu-CR] Add RegexpKuduOperationsProducer class

2016-09-10 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3883 to look at the new patch set (#5). Change subject: Add RegexpKuduOperationsProducer class .. Add RegexpKuduOperatio

[kudu-CR] Add RegexpKuduOperationsProducer class

2016-09-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add RegexpKuduOperationsProducer class .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/3346/ -- To view, visit http://gerrit.cloudera.org:80

[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 docs/release_

[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 http:/

[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 a

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

2016-09-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Reorganize and fill out release notes for 1.0 .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3344/ -- To view, visit http://gerrit.cloudera

[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 more release notes for new features in 1.0

2016-09-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add more release notes for new features in 1.0 .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3345/ -- To view, visit http://gerrit.clouder

[kudu-CR] Add more release notes for new features in 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/4370 to review the following change. Change subject: Add more release notes for new features in 1.0 .

[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 http://gerrit.

[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 htt

[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 http://gerrit.cloudera.org:8080/43

[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 http

[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 Tok

[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: http:

[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: i

[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: 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 http://gerrit.cloude

[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 http://gerrit.cloudera.org:8080/43

[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 Tok

[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 Tok

[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 2: Build Started http://104.196.14.100/job/kudu-gerrit/3340/ -- To view, visit http://gerrit.cloudera.org:8080/43

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

2016-09-10 Thread Jordan Birdsell (Code Review)
Jordan Birdsell has uploaded a new change for review. http://gerrit.cloudera.org:8080/4367 Change subject: [python] Implement Scan Token API .. [python] Implement Scan Token API First attempt at implementing the [Scan Token A

[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 http://gerrit.cloudera.org:8080/43

[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 src/ku

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

2016-09-10 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: tool: port kudu-fs_dump, remove kudu-fs_list .. Patch Set 5: (22 comments) TFTR Adar, except for one follow-up Qn about cleanup in common header, I have addresse

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

2016-09-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tool: port kudu-fs_dump, remove kudu-fs_list .. Patch Set 6: Build Started http://104.196.14.100/job/kudu-gerrit/3338/ -- To view, visit http://gerrit.cloudera.

[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] 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 http://gerrit.cloudera.org:8

[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 re

[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 docs/release_notes.ado

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

2016-09-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-236 (part 3). Enable tablet history GC by default .. KUDU-236 (part 3). Enable tablet history GC by default Change-Id: Id81d9bf0d1b33d7d2384e101d10cf

[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 docs/release_notes.ado

[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 submitted this change and it was merged. Change subject: KUDU-236 (part 2). Create randomized tablet history GC itest .. KUDU-236 (part 2). Create randomized tablet history GC itest Randomized test that performs

[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 unsubscribe,

[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 fol

[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 http:

[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 http://gerrit.cloudera.

[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! http://gerrit.cloudera.org:8080/#/c/3883/4/java/kudu-flume-sink/src/m

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

2016-09-10 Thread Will Berkeley (Code Review)
Will Berkeley 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 docs/release_notes.a

[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 http://gerrit.cloudera.org:8

[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 tablet

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

2016-09-10 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Separate tablet MM ops into separate .cc file .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/3336/ -- To view, visit http://gerrit.cloudera

[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 in

[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 docs/release_notes.adoc

[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 http://gerrit

[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 http:/

[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-2

[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 DoMergeC

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

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/3975 to look at the new patch set (#12). Change subject: KUDU-236 (part 2). Create randomized tablet history GC itest ..

[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 http:/

[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) http://gerrit.cloudera.org:8080/#/c/3975/10/src/kudu/integration-tests/