[kudu-CR](branch-0.9.x) KUDU-1477. Pending COMMIT message for failed write operation can prevent tablet startup

2016-06-23 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1477. Pending COMMIT message for failed write operation can prevent tablet startup .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/1963/

[kudu-CR](branch-0.9.x) KUDU-1477. Pending COMMIT message for failed write operation can prevent tablet startup

2016-06-23 Thread Todd Lipcon (Code Review)
Hello David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3464 to look at the new patch set (#2). Change subject: KUDU-1477. Pending COMMIT message for failed write operation can prevent tablet startup ...

[kudu-CR](branch-0.9.x) KUDU-1477. Pending COMMIT message for failed write operation can prevent tablet startup

2016-06-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1477. Pending COMMIT message for failed write operation can prevent tablet startup .. KUDU-1477. Pending COMMIT message for failed write operation ca

[kudu-CR](branch-0.9.x) KUDU-1477. Pending COMMIT message for failed write operation can prevent tablet startup

2016-06-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1477. Pending COMMIT message for failed write operation can prevent tablet startup .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.or

[kudu-CR] Allow for reserving disk space for non-Kudu processes

2016-06-23 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Allow for reserving disk space for non-Kudu processes .. Patch Set 8: Build Started http://104.196.14.100/job/kudu-gerrit/1964/ -- To view, visit http://gerrit.

[kudu-CR](branch-0.9.x) KUDU-1473: fix some tablet lock usage in CatalogManager

2016-06-23 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. KUDU-1473: fix some tablet lock usage in CatalogManager This was probably due to the refactoring

[kudu-CR](branch-0.9.x) Release notes for 0.9.1

2016-06-23 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Release notes for 0.9.1 .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/1965/ -- To view, visit http://gerrit.cloudera.org:8080/3463 To unsu

[kudu-CR](branch-0.9.x) Release notes for 0.9.1

2016-06-23 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3463 to look at the new patch set (#2). Change subject: Release notes for 0.9.1 .. Release notes for 0.9.1 Change-Id: I

[kudu-CR] Allow for reserving disk space for non-Kudu processes

2016-06-23 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Allow for reserving disk space for non-Kudu processes .. Patch Set 8: (6 comments) http://gerrit.cloudera.org:8080/#/c/3135/8/src/kudu/fs/block_manager-test.cc Fil

[kudu-CR](branch-0.9.x) Release notes for 0.9.1

2016-06-23 Thread Misty Stanley-Jones (Code Review)
Misty Stanley-Jones has posted comments on this change. Change subject: Release notes for 0.9.1 .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3463/1/docs/installation.adoc File docs/installation.adoc: Line 6

[kudu-CR](branch-0.9.x) Release notes for 0.9.1

2016-06-23 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Release notes for 0.9.1 .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/1966/ -- To view, visit http://gerrit.cloudera.org:8080/3463 To unsu

[kudu-CR](branch-0.9.x) Release notes for 0.9.1

2016-06-23 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3463 to look at the new patch set (#3). Change subject: Release notes for 0.9.1 .. Release notes for 0.9.1 Change-Id: I

[kudu-CR](branch-0.9.x) Release notes for 0.9.1

2016-06-23 Thread Misty Stanley-Jones (Code Review)
Misty Stanley-Jones has posted comments on this change. Change subject: Release notes for 0.9.1 .. Patch Set 3: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/3463 To unsubscribe, visit http://gerrit.cloudera.

[kudu-CR](branch-0.9.x) Release notes for 0.9.1

2016-06-23 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Release notes for 0.9.1 .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3463 To unsubscribe, visit http://gerrit.cloudera.o

[kudu-CR](branch-0.9.x) Release notes for 0.9.1

2016-06-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Release notes for 0.9.1 .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3463 To unsubscribe, visit http://gerrit.cloudera.org:8080/se

[kudu-CR](branch-0.9.x) Release notes for 0.9.1

2016-06-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Release notes for 0.9.1 .. Release notes for 0.9.1 Change-Id: I797980d105e0e1000420aedf58e094f9505053c6 Reviewed-on: http://gerrit.cloudera.org:8080/3463

[kudu-CR] lock manager: fix TSAN error in LockTable::Resize()

2016-06-23 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: lock_manager: fix TSAN error in LockTable::Resize() .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/1967/ -- To view, visit http://gerrit.cl

[kudu-CR] lock manager: fix TSAN error in LockTable::Resize()

2016-06-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/3469 Change subject: lock_manager: fix TSAN error in LockTable::Resize() .. lock_manager: fix TSAN error in LockTable::Resize() This function w

[kudu-CR] Release notes for 0.9.1

2016-06-23 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Release notes for 0.9.1 .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/1968/ -- To view, visit http://gerrit.cloudera.org:8080/3470 To unsu

[kudu-CR] Release notes for 0.9.1

2016-06-23 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, Misty Stanley-Jones, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3470 to review the following change. Change subject: Release notes for 0.9.1 .. Release notes f

[kudu-CR] lock manager: fix TSAN error in LockTable::Resize()

2016-06-23 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: lock_manager: fix TSAN error in LockTable::Resize() .. Patch Set 1: Code-Review+2 Which annotation was hiding this previously? -- To view, visit http://gerrit.clo

[kudu-CR] lock manager: fix TSAN error in LockTable::Resize()

2016-06-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: lock_manager: fix TSAN error in LockTable::Resize() .. Patch Set 1: We had a suppression on ScopedRowLock::Release originally (since we first introduced TSAN a co

[kudu-CR] lock manager: fix TSAN error in LockTable::Resize()

2016-06-23 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: lock_manager: fix TSAN error in LockTable::Resize() .. Patch Set 1: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/1969/ -- To view, visit http:/

[kudu-CR] lock manager: fix TSAN error in LockTable::Resize()

2016-06-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: lock_manager: fix TSAN error in LockTable::Resize() .. lock_manager: fix TSAN error in LockTable::Resize() This function was doing a non-atomic read of it

[kudu-CR] Allow for reserving disk space for non-Kudu processes

2016-06-23 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Allow for reserving disk space for non-Kudu processes .. Patch Set 5: (5 comments) http://gerrit.cloudera.org:8080/#/c/3135/8/src/kudu/fs/block_manager-test.cc Fil

[kudu-CR] Allow for reserving disk space for non-Kudu processes

2016-06-23 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Allow for reserving disk space for non-Kudu processes .. Patch Set 9: Build Started http://104.196.14.100/job/kudu-gerrit/1970/ -- To view, visit http://gerrit.

[kudu-CR] Allow for reserving disk space for non-Kudu processes

2016-06-23 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3135 to look at the new patch set (#9). Change subject: Allow for reserving disk space for non-Kudu processes .. Allow f

[kudu-CR] Release notes for 0.9.1

2016-06-23 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Release notes for 0.9.1 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3470 To unsubscribe, visit http://gerrit.cloudera.o

[kudu-CR](branch-0.9.x) Bump version to 0.9.1

2016-06-23 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3473 to review the following change. Change subject: Bump version to 0.9.1 .. Bump version to 0.9.1 Change-Id: Ied9

[kudu-CR](branch-0.9.x) Bump version to 0.9.1

2016-06-23 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Bump version to 0.9.1 .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/1971/ -- To view, visit http://gerrit.cloudera.org:8080/3473 To unsubs

[kudu-CR](branch-0.9.x) Bump version to 0.9.1

2016-06-23 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Bump version to 0.9.1 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3473 To unsubscribe, visit http://gerrit.cloudera.org

[kudu-CR](branch-0.9.x) Bump version to 0.9.1

2016-06-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Bump version to 0.9.1 .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3473 To unsubscribe, visit http://gerrit.cloudera.org:8080/sett

[kudu-CR](branch-0.9.x) Bump version to 0.9.1

2016-06-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Bump version to 0.9.1 .. Bump version to 0.9.1 Change-Id: Ied911c99b45f9bdae20d14f0ecbe2ae9010f1476 Reviewed-on: http://gerrit.cloudera.org:8080/3473 Revi

[kudu-CR] Allow for reserving disk space for non-Kudu processes

2016-06-23 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Allow for reserving disk space for non-Kudu processes .. Patch Set 9: Code-Review+2 Leaving unmerged in case Todd wants to rereview. -- To view, visit http://gerr

[kudu-CR] Allow for reserving disk space for non-Kudu processes

2016-06-23 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Allow for reserving disk space for non-Kudu processes .. Patch Set 9: Verified+1 Overriding apparent Java test flake. Filed https://issues.apache.org/jira/browse/K

[kudu-CR] Release notes for 0.9.1

2016-06-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Release notes for 0.9.1 .. Release notes for 0.9.1 Change-Id: I797980d105e0e1000420aedf58e094f9505053c6 Reviewed-on: http://gerrit.cloudera.org:8080/3463

[kudu-CR] alter schema transaction: don't crash in ToString() when no timestamp

2016-06-23 Thread Adar Dembo (Code Review)
Hello David Ribeiro Alves, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3475 to review the following change. Change subject: alter_schema_transaction: don't crash in ToString() when no timestamp

[kudu-CR] alter schema transaction: don't crash in ToString() when no timestamp

2016-06-23 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: alter_schema_transaction: don't crash in ToString() when no timestamp .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/1972/ -- To view, vis

[kudu-CR] alter schema transaction: don't crash in ToString() when no timestamp

2016-06-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: alter_schema_transaction: don't crash in ToString() when no timestamp .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/347

[kudu-CR] alter schema transaction: don't crash in ToString() when no timestamp

2016-06-23 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: alter_schema_transaction: don't crash in ToString() when no timestamp .. alter_schema_transaction: don't crash in ToString() when no timestamp Saw this in

[kudu-CR] Allow for reserving disk space for non-Kudu processes

2016-06-23 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Allow for reserving disk space for non-Kudu processes .. Patch Set 9: Hey Mike, are you planning on doing a follow-up patch with some documentation? Also,

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-23 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 11: Build Started http://104.196.14.100/job/kudu-gerrit/1973/ -- To view, visit

[kudu-CR] Allow for reserving disk space for non-Kudu processes

2016-06-23 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Allow for reserving disk space for non-Kudu processes .. Patch Set 9: JD: I'm happy to add docs as a follow up. I've actually not written much documentation so I'

[kudu-CR] catalog manager: prevent spurious dirty callbacks from crashing the process

2016-06-23 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: catalog_manager: prevent spurious dirty callbacks from crashing the process .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/3465/2/src/kudu/maste

[kudu-CR] Allow for reserving disk space for non-Kudu processes

2016-06-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Allow for reserving disk space for non-Kudu processes .. Patch Set 9: Any particular areas you think I should take a close look at? Otherwise happy to trust that

[kudu-CR] catalog manager: prevent spurious dirty callbacks from crashing the process

2016-06-23 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: catalog_manager: prevent spurious dirty callbacks from crashing the process .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/3465/2/src/kudu/maste

[kudu-CR] catalog manager: prevent spurious dirty callbacks from crashing the process

2016-06-23 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3465 to look at the new patch set (#3). Change subject: catalog_manager: prevent spurious dirty callbacks from crashing the process

[kudu-CR] catalog manager: prevent spurious dirty callbacks from crashing the process

2016-06-23 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: catalog_manager: prevent spurious dirty callbacks from crashing the process .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/1974/ -- To vie

[kudu-CR] Allow for reserving disk space for non-Kudu processes

2016-06-23 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Allow for reserving disk space for non-Kudu processes .. Patch Set 9: Code-Review+1 > JD: > > I'm happy to add docs as a follow up. I've actually not wri

[kudu-CR] Allow for reserving disk space for non-Kudu processes

2016-06-23 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Allow for reserving disk space for non-Kudu processes .. Patch Set 9: > Any particular areas you think I should take a close look at? Nothing specific, I just want

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. KUDU-1398 CFile index blocks can store shortest separating prefix (No changes: resub

[kudu-CR] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 11: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3304 To unsubs

[kudu-CR] Allow for reserving disk space for non-Kudu processes

2016-06-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Allow for reserving disk space for non-Kudu processes .. Patch Set 9: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/3135 To unsubscribe, visit h

[kudu-CR] Add required Debian version to installation page

2016-06-23 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add required Debian version to installation page .. Patch Set 1: -Code-Review JD, are you ok with pushing this now with your recent deb 8 testing? -- To view, vi

[kudu-CR] catalog manager: prevent spurious dirty callbacks from crashing the process

2016-06-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: catalog_manager: prevent spurious dirty callbacks from crashing the process .. Patch Set 3: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/3465

[kudu-CR] catalog manager: prevent spurious dirty callbacks from crashing the process

2016-06-23 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: catalog_manager: prevent spurious dirty callbacks from crashing the process .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3465 T

[kudu-CR] catalog manager: prevent spurious dirty callbacks from crashing the process

2016-06-23 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: catalog_manager: prevent spurious dirty callbacks from crashing the process .. catalog_manager: prevent spurious dirty callbacks from crashing the process

[kudu-CR] Allow for reserving disk space for non-Kudu processes

2016-06-23 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: Allow for reserving disk space for non-Kudu processes .. Allow for reserving disk space for non-Kudu processes Adds gflags to reserve disk space such that

[kudu-CR](gh-pages) Add blog post about multi-master support in Kudu 1.0

2016-06-23 Thread Adar Dembo (Code Review)
Hello Jean-Daniel Cryans, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3476 to review the following change. Change subject: Add blog post about multi-master support in Kudu 1.0 ..

[kudu-CR] [java-client] refactor AsyncKuduSession

2016-06-23 Thread Dan Burkert (Code Review)
Hello Jean-Daniel Cryans, Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3477 to review the following change. Change subject: [java-client] refactor AsyncKuduSession .. [java-cl

[kudu-CR] [java-client] refactor AsyncKuduSession

2016-06-23 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java-client] refactor AsyncKuduSession .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/1976/ -- To view, visit http://gerrit.cloudera.org:8

[kudu-CR] KUDU-1309: [java client] support tables with non-covering partition-key ranges

2016-06-23 Thread Dan Burkert (Code Review)
Hello Jean-Daniel Cryans, Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3388 to look at the new patch set (#3). Change subject: KUDU-1309: [java client] support tables with non-covering partition-key ranges .

[kudu-CR] KUDU-1309: [java client] support tables with non-covering partition-key ranges

2016-06-23 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1309: [java client] support tables with non-covering partition-key ranges .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/1975/ -- To

[kudu-CR] KUDU-1309: [java client] support tables with non-covering partition-key ranges

2016-06-23 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1309: [java client] support tables with non-covering partition-key ranges .. Patch Set 2: (32 comments) http://gerrit.cloudera.org:8080/#/c/3388/2/java/kudu

[kudu-CR] Add required Debian version to installation page

2016-06-23 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add required Debian version to installation page .. Patch Set 1: > JD, are you ok with pushing this now with your recent deb 8 > testing? Yes. -- To vie

[kudu-CR](gh-pages) Add blog post about multi-master support in Kudu 1.0

2016-06-23 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add blog post about multi-master support in Kudu 1.0 .. Patch Set 1: Code-Review+2 Good stuff! -- To view, visit http://gerrit.cloudera.org:8080/3476 To u