[kudu-CR] Avoid missing 'override' keyword warnings in raft consensus-test.cc

2016-06-03 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Avoid missing 'override' keyword warnings in raft_consensus-test.cc .. Patch Set 1: https://github.com/google/googletest/issues/533 has some discussion here --

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

2016-06-03 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [WIP] KUDU-1398 CFile index blocks can store shortest separating prefix .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3304/2//COMMIT_MSG

[kudu-CR] KUDU-1469. Fix handling of fully-deduped requests after a leader change

2016-06-07 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3228 to look at the new patch set (#4). Change subject: KUDU-1469. Fix handling of fully-deduped requests after a leader change

[kudu-CR] KUDU-1473: fix some tablet lock usage in CatalogManager

2016-06-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3309/2//COMMIT_MSG Commit Message: Line

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

2016-06-07 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 4: (15 comments) http://gerrit.cloudera.org:8080/#/c/3304/4//COMMIT_MSG Commit

[kudu-CR] WIP: KUDU-1466: improve error message when writes fail at TS

2016-06-07 Thread Todd Lipcon (Code Review)
Hello Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3326 to review the following change. Change subject: WIP: KUDU-1466: improve error message when writes fail at TS .. WIP:

[kudu-CR] KUDU-1446 Consider data types in predicate evaluation order

2016-06-03 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1446 Consider data types in predicate evaluation order .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3298 To unsubscribe,

[kudu-CR] KUDU-1467. Fix incorrect bootstrap replay issue with UPSERT

2016-05-24 Thread Todd Lipcon (Code Review)
Hello David Ribeiro Alves, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3208 to review the following change. Change subject: KUDU-1467. Fix incorrect bootstrap replay issue with UPSERT ..

[kudu-CR](branch-0.9.x) java: use truncated randomized exponential backoff for retries

2016-05-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: java: use truncated randomized exponential backoff for retries .. java: use truncated randomized exponential backoff for retries This changes the Java

[kudu-CR] KUDU-1444. Get resource metrics of a scan.

2016-05-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1444. Get resource metrics of a scan. .. Patch Set 5: (7 comments) http://gerrit.cloudera.org:8080/#/c/3013/5/src/kudu/client/client.h File

[kudu-CR] KUDU-749 (part 2): avoid O(n^2) behavior when compacting deltas

2016-05-25 Thread Todd Lipcon (Code Review)
Hello David Ribeiro Alves, Mike Percy, Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3221 to look at the new patch set (#2). Change subject: KUDU-749 (part 2): avoid O(n^2) behavior when compacting deltas

[kudu-CR] KUDU-1385 Part 1: Modify documentation landing page

2016-06-03 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1385 Part 1: Modify documentation landing page .. KUDU-1385 Part 1: Modify documentation landing page Rename introduction.html to index.html and get

[kudu-CR] Add unique id generation to the client

2016-05-30 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add unique id generation to the client .. Patch Set 14: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3077 To unsubscribe, visit

[kudu-CR] Add a RpcContext::RespondFailure() method

2016-05-30 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add a RpcContext::RespondFailure() method .. Patch Set 6: (2 comments) wonder whether it would be a clearer API to instead have a flag like

[kudu-CR] KUDU-1444. Get resource metrics of a scan.

2016-05-26 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1444. Get resource metrics of a scan. .. Patch Set 6: OK. The code now looks good, but missing a test. Sorry, I should have mentioned that previously. Maybe

[kudu-CR](gh-pages) Add weekly update for 06/01

2016-06-01 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add weekly update for 06/01 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3269 To unsubscribe, visit

[kudu-CR](branch-0.9.x) Disable tracing for tablet bootstrap during remote bootstrap

2016-06-01 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/3267 Change subject: Disable tracing for tablet bootstrap during remote bootstrap .. Disable tracing for tablet bootstrap during remote

[kudu-CR](branch-0.9.x) Add python/pytest.ini in RAT exclude

2016-06-01 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Add python/pytest.ini in RAT exclude .. Add python/pytest.ini in RAT exclude Needed just for branch-0.9.x. Change-Id:

[kudu-CR](branch-0.9.x) Make 0.9.0 not SNAPSHOT for release

2016-06-01 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Make 0.9.0 not SNAPSHOT for release .. Make 0.9.0 not SNAPSHOT for release Change-Id: I912336be4cce91f8708673b32e6f18ff0022f0cf Reviewed-on:

[kudu-CR](branch-0.9.x) Make 0.9.0 not SNAPSHOT for release

2016-06-01 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Make 0.9.0 not SNAPSHOT for release .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3261 To unsubscribe, visit

[kudu-CR] KUDU-1444. Get resource metrics of a scan.

2016-05-30 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1444. Get resource metrics of a scan. .. Patch Set 8: Looks like this needs a manual rebase -- To view, visit http://gerrit.cloudera.org:8080/3013 To

[kudu-CR] KUDU-1444. Get resource metrics of a scan.

2016-05-30 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1444. Get resource metrics of a scan. .. Patch Set 8: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3013 To unsubscribe, visit

[kudu-CR] log: Mark allocation finished even if allocation had an error

2016-05-30 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: log: Mark allocation finished even if allocation had an error .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3234/2/src/kudu/consensus/log.cc

[kudu-CR] Specify guaranteed semantics of GetTableLocations RPC

2016-05-30 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Specify guaranteed semantics of GetTableLocations RPC .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3240/1/src/kudu/master/catalog_manager.cc

[kudu-CR] Allow to set RequestId in the RPC RequestHeader

2016-05-30 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Allow to set RequestId in the RPC RequestHeader .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/3179/3/src/kudu/rpc/rpc_controller.h File

[kudu-CR] Fix stray memory writes due to tcmalloc profiling

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Fix stray memory writes due to tcmalloc profiling .. Fix stray memory writes due to tcmalloc profiling This fixes an issue that has been causing frequent

[kudu-CR] Fix stray memory writes due to tcmalloc profiling

2016-06-22 Thread Todd Lipcon (Code Review)
Hello David Ribeiro Alves, Mike Percy, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3445 to review the following change. Change subject: Fix stray memory writes due to tcmalloc profiling

[kudu-CR] Add a ResultTracker class that will track server side results

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add a ResultTracker class that will track server side results .. Patch Set 12: (25 comments)

[kudu-CR] log: Mark allocation finished even if allocation had an error

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: log: Mark allocation finished even if allocation had an error .. Patch Set 4: Code-Review+1 (1 comment)

[kudu-CR] tablet: fix sloppiness around atomic mutation lists

2016-06-22 Thread Todd Lipcon (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3410 to look at the new patch set (#2). Change subject: tablet: fix sloppiness around atomic mutation lists

[kudu-CR](gh-pages) Add weekly update for 06/21

2016-06-21 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add weekly update for 06/21 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3415 To unsubscribe, visit

[kudu-CR] Allow to set RequestId in the RPC RequestHeader

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Allow to set RequestId in the RPC RequestHeader .. Patch Set 9: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3179 To unsubscribe, visit

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

2016-06-22 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 9: Looks like a small compile error here -- To view, visit

[kudu-CR] KUDU-1491. Address TSAN warning for compaction

2016-06-22 Thread Todd Lipcon (Code Review)
Hello Mike Percy, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3454 to review the following change. Change subject: KUDU-1491. Address TSAN warning for compaction .. KUDU-1491. Address

[kudu-CR](branch-0.9.x) update dist-test configuration for new client.py location

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/3459 Change subject: update dist-test configuration for new client.py location .. update dist-test configuration for new client.py location

[kudu-CR] KUDU-1477. Pending COMMIT message for failed write operation can prevent tablet startup

2016-06-22 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 1: (5 comments)

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

2016-06-22 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/3458 to review the following change. Change subject: Bump version to 0.9.1-SNAPSHOT .. Bump version to

[kudu-CR] KUDU-1491. Address TSAN warning for compaction

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1491. Address TSAN warning for compaction .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3454/1/src/kudu/tablet/compaction.cc File

[kudu-CR] KUDU-1477. Pending COMMIT message for failed write operation can prevent tablet startup

2016-06-22 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3321 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-1469. Fix handling of fully-deduped requests after a leader change

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/3455 Change subject: KUDU-1469. Fix handling of fully-deduped requests after a leader change .. KUDU-1469. Fix handling of fully-deduped

[kudu-CR] Integrate the request tracker with the client

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Integrate the request tracker with the client .. Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/3080/8/src/kudu/rpc/retriable_rpc.h File

[kudu-CR] KUDU-1469. Fix handling of fully-deduped requests after a leader change

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1469. Fix handling of fully-deduped requests after a leader change .. Patch Set 4: (1 comment)

[kudu-CR] Clarify the difference between 'call id' and 'request id'

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Clarify the difference between 'call_id' and 'request_id' .. Patch Set 6: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3409 To

[kudu-CR] Clarify the difference between 'call id' and 'request id'

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Clarify the difference between 'call_id' and 'request_id' .. Clarify the difference between 'call_id' and 'request_id' This adds a bit more information

[kudu-CR](branch-0.9.x) Fix stray memory writes due to tcmalloc profiling

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/3456 Change subject: Fix stray memory writes due to tcmalloc profiling .. Fix stray memory writes due to tcmalloc profiling This fixes an

[kudu-CR] KUDU-1491. Address TSAN warning for compaction

2016-06-22 Thread Todd Lipcon (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3454 to look at the new patch set (#2). Change subject: KUDU-1491. Address TSAN warning for compaction ..

[kudu-CR] Integrate the request tracker with the client

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Integrate the request tracker with the client .. Patch Set 8: (10 comments) http://gerrit.cloudera.org:8080/#/c/3080/8//COMMIT_MSG Commit Message: Line 15: any

[kudu-CR](branch-0.9.x) [java-client] RPC timeout may sometimes be reported as max attempts violation

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/3457 Change subject: [java-client] RPC timeout may sometimes be reported as max attempts violation .. [java-client] RPC timeout may sometimes

[kudu-CR] Avoid missing 'override' keyword warnings in raft consensus-test.cc

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Avoid missing 'override' keyword warnings in raft_consensus-test.cc .. Patch Set 2: (1 comment)

[kudu-CR] Integrate the ResultTracker into the rpc subsystem and add a test

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Integrate the ResultTracker into the rpc subsystem and add a test .. Patch Set 11: (9 comments)

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

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/3464 Change subject: KUDU-1477. Pending COMMIT message for failed write operation can prevent tablet startup .. KUDU-1477. Pending COMMIT

[kudu-CR](gh-pages) Control how content is cached on the Kudu web site

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Control how content is cached on the Kudu web site .. Patch Set 1: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3462 To

[kudu-CR] KUDU-1491. Address TSAN warning for compaction

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1491. Address TSAN warning for compaction .. KUDU-1491. Address TSAN warning for compaction This adds a missing memory barrier that was exposed when

[kudu-CR](branch-0.9.x) [java-client] RPC timeout may sometimes be reported as max attempts violation

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: [java-client] RPC timeout may sometimes be reported as max attempts violation .. [java-client] RPC timeout may sometimes be reported as max attempts

[kudu-CR](branch-0.9.x) Fix stray memory writes due to tcmalloc profiling

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Fix stray memory writes due to tcmalloc profiling .. Fix stray memory writes due to tcmalloc profiling This fixes an issue that has been causing frequent

[kudu-CR](branch-0.9.x) Fix stray memory writes due to tcmalloc profiling

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Fix stray memory writes due to tcmalloc profiling .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3456 To unsubscribe, visit

[kudu-CR](branch-0.9.x) KUDU-1469. Fix handling of fully-deduped requests after a leader change

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1469. Fix handling of fully-deduped requests after a leader change .. KUDU-1469. Fix handling of fully-deduped requests after a leader change This

[kudu-CR](gh-pages) Control how content is cached on the Kudu web site

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Control how content is cached on the Kudu web site .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3462/1/.htaccess File .htaccess: Line 1: #

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

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

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

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Bump version to 0.9.1-SNAPSHOT .. Bump version to 0.9.1-SNAPSHOT Change-Id: I472c2999c0b13a2c046366a9e1ad7474eedf68d0 Reviewed-on:

[kudu-CR](branch-0.9.x) update dist-test configuration for new client.py location

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: update dist-test configuration for new client.py location .. update dist-test configuration for new client.py location client.py has been moved to

[kudu-CR](branch-0.9.x) update dist-test configuration for new client.py location

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: update dist-test configuration for new client.py location .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3459 To unsubscribe,

[kudu-CR](branch-0.9.x) KUDU-1469. Fix handling of fully-deduped requests after a leader change

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1469. Fix handling of fully-deduped requests after a leader change .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3455 To

[kudu-CR](gh-pages) Control how content is cached on the Kudu web site

2016-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Control how content is cached on the Kudu web site .. Control how content is cached on the Kudu web site Change-Id:

[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

[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] KUDU-1398 CFile index blocks can store shortest separating prefix

2016-06-20 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 4: Sorry for the delay on this, Will. I was on PTO the last couple of weeks. Hope

[kudu-CR] tablet: fix sloppiness around atomic mutation lists

2016-06-20 Thread Todd Lipcon (Code Review)
Hello David Ribeiro Alves, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3410 to review the following change. Change subject: tablet: fix sloppiness around atomic mutation lists ..

[kudu-CR] tsan: remove two old suppressions

2016-06-20 Thread Todd Lipcon (Code Review)
Hello Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3411 to review the following change. Change subject: tsan: remove two old suppressions .. tsan: remove two old suppressions

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

[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

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

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

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

[kudu-CR](gh-pages) Fix .gitignore

2016-06-24 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Fix .gitignore .. Patch Set 1: this seems kind of fragile.. why not just use a different checkout for this branch? -- To view, visit

[kudu-CR](gh-pages) Fix .gitignore

2016-06-24 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Fix .gitignore .. Patch Set 1: It's just trying to prevent you from tryig to 'publish' when in fact you forgot to 'git add' something. Could add a

[kudu-CR](gh-pages) Fix .gitignore

2016-06-24 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Fix .gitignore .. Patch Set 1: Although I guess I should add that the reason it does this is that jekyll will see all your untracked files, think they are

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

[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](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

[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

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

[kudu-CR] Add a ToString() method to Proxy

2016-06-27 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add a ToString() method to Proxy .. Patch Set 1: would be nice to add a new assertion to one of the RPC layer tests which calls this -- To view, visit

[kudu-CR] tablet: change default bloom filter FP rate to 0.01%

2016-06-27 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/3517 to review the following change. Change subject: tablet: change default bloom filter FP rate to 0.01% ..

[kudu-CR] Implement kudu::optional replacement for boost::optional

2016-06-27 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Implement kudu::optional replacement for boost::optional .. Patch Set 1: yea, ASAN does insert poisoning between variables on the stack:

[kudu-CR](gh-pages) Add 6/27 weekly update

2016-06-27 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Add 6/27 weekly update .. Add 6/27 weekly update Change-Id: I6ce4bf6e02a5481bf3e032d3345d7afc93c244a2 Reviewed-on: http://gerrit.cloudera.org:8080/3515

[kudu-CR](gh-pages) Add 6/27 weekly update

2016-06-27 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add 6/27 weekly update .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3515 To unsubscribe, visit

[kudu-CR](gh-pages) Add 6/27 weekly update

2016-06-27 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/3515 to review the following change. Change subject: Add 6/27 weekly update .. Add 6/27 weekly update Change-Id:

[kudu-CR] Fix flakiness in RaftConsensusITest.TestMasterNotifiedOnConfigChange

2016-06-27 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Fix flakiness in RaftConsensusITest.TestMasterNotifiedOnConfigChange .. Patch Set 1: A java test failed due to https://issues.apache.org/jira/browse/KUDU-1496

[kudu-CR] Fix flakiness in RaftConsensusITest.TestMasterNotifiedOnConfigChange

2016-06-27 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Fix flakiness in RaftConsensusITest.TestMasterNotifiedOnConfigChange .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3511 To

[kudu-CR] locks: add new read-write mutex

2016-06-27 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: locks: add new read-write mutex .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/3496/2/src/kudu/util/rw_mutex.cc File src/kudu/util/rw_mutex.cc:

[kudu-CR] locks: switch from boost::shared lock to kudu::shared lock

2016-06-27 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: locks: switch from boost::shared_lock to kudu::shared_lock .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3498 To unsubscribe,

[kudu-CR] Fix flakiness of RaftConsensusITest.TestLogIOErrorIsFatal

2016-06-27 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Fix flakiness of RaftConsensusITest.TestLogIOErrorIsFatal .. Fix flakiness of RaftConsensusITest.TestLogIOErrorIsFatal This test was flaky because it was

[kudu-CR] locks: stop using errno around base::NumCPUs and base::MaxCPUIndex

2016-06-27 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: locks: stop using errno around base::NumCPUs and base::MaxCPUIndex .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3495 To

[kudu-CR] tablet: change default bloom filter FP rate to 0.01%

2016-06-27 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3517 to look at the new patch set (#2). Change subject: tablet: change default bloom filter FP rate to 0.01%

[kudu-CR] Avoid missing 'override' keyword warnings in raft consensus-test.cc

2016-06-27 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Avoid missing 'override' keyword warnings in raft_consensus-test.cc .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3289 To

[kudu-CR] Add a Barrier implementation and use it in cbtree-test

2016-06-24 Thread Todd Lipcon (Code Review)
Hello Dan Burkert, Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3494 to review the following change. Change subject: Add a Barrier implementation and use it in cbtree-test ..

  1   2   3   >