[kudu-CR] env: rename writable file use fsync to env file use fsync

2016-12-09 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Mike Percy, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5458 to review the following change. Change subject: env: rename writable_file_use_fsync to env_file_use_fsync

[kudu-CR] [delete table-test] parameterized DeleteTableWhileScanInProgress

2016-12-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/5455 Change subject: [delete_table-test] parameterized DeleteTableWhileScanInProgress .. [delete_table-test] parameterized

[kudu-CR] [spark] KUDU-1641 Pushdown SparkSQL In predicates

2016-12-09 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5425 to look at the new patch set (#3). Change subject: [spark] KUDU-1641 Pushdown SparkSQL In predicates .. [spark]

[kudu-CR] WIP: bug in exactly-once during tablet bootstrap

2016-12-09 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: WIP: bug in exactly-once during tablet bootstrap .. Patch Set 2: I figured out the bug. Will post a fix soon -- To view, visit

[kudu-CR](branch-1.2.x) KUDU-1798: [env] File manager broken on OS X 10.11

2016-12-09 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: KUDU-1798: [env] File manager broken on OS X 10.11 .. KUDU-1798: [env] File manager broken on OS X 10.11 Env::IncreaseOpenFileLimit() relies upon few

[kudu-CR] MiniKuduClusterBuilder.build should clean up state when initialization fails

2016-12-09 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: MiniKuduClusterBuilder.build should clean up state when initialization fails .. MiniKuduClusterBuilder.build should clean up state when initialization

[kudu-CR](branch-1.2.x) csd: two minor changes

2016-12-09 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: csd: two minor changes .. csd: two minor changes Revert the memory limit scaleFactor to the default value (1.0). 1.3 is recommended for Java processes;

[kudu-CR](branch-1.2.x) KUDU-1799: AsyncKuduClient.discoverTablets has very expensive logging

2016-12-09 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1799: AsyncKuduClient.discoverTablets has very expensive logging .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] MiniKuduClusterBuilder.build should clean up state when initialization fails

2016-12-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: MiniKuduClusterBuilder.build should clean up state when initialization fails .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5448

[kudu-CR](branch-1.2.x) csd: two minor changes

2016-12-09 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: csd: two minor changes .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5450 To unsubscribe, visit

[kudu-CR](branch-1.2.x) csd: two minor changes

2016-12-09 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/5450 Change subject: csd: two minor changes .. csd: two minor changes Revert the memory limit scaleFactor to the default value (1.0).

[kudu-CR](branch-1.2.x) KUDU-1798: [env] File manager broken on OS X 10.11

2016-12-09 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded a new change for review. http://gerrit.cloudera.org:8080/5452 Change subject: KUDU-1798: [env] File manager broken on OS X 10.11 .. KUDU-1798: [env] File manager broken on OS X 10.11

[kudu-CR](branch-1.2.x) KUDU-1799: AsyncKuduClient.discoverTablets has very expensive logging

2016-12-09 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded a new change for review. http://gerrit.cloudera.org:8080/5451 Change subject: KUDU-1799: AsyncKuduClient.discoverTablets has very expensive logging .. KUDU-1799: AsyncKuduClient.discoverTablets has

[kudu-CR] MiniKuduClusterBuilder.build should clean up state when initialization fails

2016-12-09 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: MiniKuduClusterBuilder.build should clean up state when initialization fails .. Patch Set 2: (1 comment)

[kudu-CR] MiniKuduClusterBuilder.build should clean up state when initialization fails

2016-12-09 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5448 to look at the new patch set (#3). Change subject: MiniKuduClusterBuilder.build should clean up state when initialization fails

[kudu-CR] csd: two minor changes

2016-12-09 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: csd: two minor changes .. csd: two minor changes Revert the memory limit scaleFactor to the default value (1.0). 1.3 is recommended for Java

[kudu-CR] csd: two minor changes

2016-12-09 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: csd: two minor changes .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5435 To unsubscribe, visit

[kudu-CR] MiniKuduClusterBuilder.build should clean up state when initialization fails

2016-12-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: MiniKuduClusterBuilder.build should clean up state when initialization fails .. Patch Set 2: (1 comment)

[kudu-CR] csd: two minor changes

2016-12-09 Thread Anonymous Coward (Code Review)
Anonymous Coward #265 has posted comments on this change. Change subject: csd: two minor changes .. Patch Set 2: Code-Review+1 (1 comment) http://gerrit.cloudera.org:8080/#/c/5435/2//COMMIT_MSG Commit Message: Line 20: To

[kudu-CR] MiniKuduClusterBuilder.build should clean up state when initialization fails

2016-12-09 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded a new patch set (#2). Change subject: MiniKuduClusterBuilder.build should clean up state when initialization fails .. MiniKuduClusterBuilder.build should clean up state when initialization fails The

[kudu-CR] MiniKuduClusterBuilder.build should clean up state when initialization fails

2016-12-09 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/5448 to review the following change. Change subject: MiniKuduClusterBuilder.build should clean up state when initialization fails

[kudu-CR] KUDU-1798: [env] File manager broken on OS X 10.11

2016-12-09 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: KUDU-1798: [env] File manager broken on OS X 10.11 .. KUDU-1798: [env] File manager broken on OS X 10.11 Env::IncreaseOpenFileLimit() relies upon few

[kudu-CR] KUDU-1798: [env] File manager broken on OS X 10.11

2016-12-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1798: [env] File manager broken on OS X 10.11 .. Patch Set 6: Code-Review+2 (1 comment)

[kudu-CR] KUDU-1798: [env] File manager broken on OS X 10.11

2016-12-09 Thread Dan Burkert (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5438 to look at the new patch set (#6). Change subject: KUDU-1798: [env] File manager broken on OS X 10.11

[kudu-CR] log-rolling-itest: remove unused mini cluster option

2016-12-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: log-rolling-itest: remove unused mini cluster option .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5444 To unsubscribe, visit

[kudu-CR] log-rolling-itest: remove unused mini cluster option

2016-12-09 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: log-rolling-itest: remove unused mini cluster option .. log-rolling-itest: remove unused mini cluster option The option does not take effect since the

[kudu-CR] WIP: bug in exactly-once during tablet bootstrap

2016-12-09 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: WIP: bug in exactly-once during tablet bootstrap .. Patch Set 2: After digging around a bit more. I think the problem is a bit more insidious than us

[kudu-CR] csd: two minor changes

2016-12-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: csd: two minor changes .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5435/1//COMMIT_MSG Commit Message: Line 23: So this probably makes the

[kudu-CR] KUDU-1798: [env] File manager broken on OS X 10.11

2016-12-09 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5438 to look at the new patch set (#5). Change subject: KUDU-1798: [env] File manager broken on OS X 10.11 ..

[kudu-CR] KUDU-1798: [env] File manager broken on OS X 10.11

2016-12-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1798: [env] File manager broken on OS X 10.11 .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/5438/4/src/kudu/util/env_posix.cc File

[kudu-CR] KUDU-1799: AsyncKuduClient.discoverTablets has very expensive logging

2016-12-09 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: KUDU-1799: AsyncKuduClient.discoverTablets has very expensive logging .. KUDU-1799: AsyncKuduClient.discoverTablets has very expensive logging

[kudu-CR] KUDU-1798: [env] File manager broken on OS X 10.11

2016-12-09 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5438 to look at the new patch set (#4). Change subject: KUDU-1798: [env] File manager broken on OS X 10.11 ..

[kudu-CR] KUDU-1798: [env] File manager broken on OS X 10.11

2016-12-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1798: [env] File manager broken on OS X 10.11 .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/5438/3/src/kudu/util/env-test.cc File

[kudu-CR] KUDU-1798: [env] File manager broken on OS X 10.11

2016-12-09 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1798: [env] File manager broken on OS X 10.11 .. Patch Set 3: Hey Dinesh, Adar and I looked into this and isolated it to just 10.11. The original code you

[kudu-CR] KUDU-1798: [env] File manager broken on OS X 10.11

2016-12-09 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5438 to look at the new patch set (#3). Change subject: KUDU-1798: [env] File manager broken on OS X 10.11 ..

[kudu-CR] KUDU-1595 and KUDU-1642: IS NOT NULL and IS NULL predicates

2016-12-09 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5439 to look at the new patch set (#2). Change subject: KUDU-1595 and KUDU-1642: IS NOT NULL and IS NULL predicates ..

[kudu-CR] KUDU-1799: AsyncKuduClient.discoverTablets has very expensive logging

2016-12-09 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: KUDU-1799: AsyncKuduClient.discoverTablets has very expensive logging .. Patch Set 1: Casting a non-binding vote for inclusion in 1.2 :) -- To view, visit

[kudu-CR] KUDU-1799: AsyncKuduClient.discoverTablets has very expensive logging

2016-12-09 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: KUDU-1799: AsyncKuduClient.discoverTablets has very expensive logging .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/5443 To

[kudu-CR] [spark] KUDU-1641 Pushdown SparkSQL In predicates

2016-12-09 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: [spark] KUDU-1641 Pushdown SparkSQL In predicates .. Patch Set 2: > Until we implement a max in the Kudu server I don't think it makes > sense to limit the

[kudu-CR] KUDU-1595 and KUDU-1642: IS NOT NULL and IS NULL predicates

2016-12-09 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-1595 and KUDU-1642: IS NOT NULL and IS NULL predicates .. Patch Set 1: (11 comments)

[kudu-CR] KUDU-1799: AsyncKuduClient.discoverTablets has very expensive logging

2016-12-09 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1799: AsyncKuduClient.discoverTablets has very expensive logging .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-1799: AsyncKuduClient.discoverTablets has very expensive logging

2016-12-09 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1799: AsyncKuduClient.discoverTablets has very expensive logging .. Patch Set 1: Nominating for inclusion in 1.2. -- To view, visit

[kudu-CR] KUDU-1799: AsyncKuduClient.discoverTablets has very expensive logging

2016-12-09 Thread Dan Burkert (Code Review)
Hello Jean-Daniel Cryans, Matthew Jacobs, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5443 to review the following change. Change subject: KUDU-1799: AsyncKuduClient.discoverTablets has very expensive logging

[kudu-CR] KUDU-1798: [env] File manager broken on OS X

2016-12-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1798: [env] File manager broken on OS X .. Patch Set 2: (5 comments) http://gerrit.cloudera.org:8080/#/c/5438/2//COMMIT_MSG Commit Message: PS2, Line 21:

[kudu-CR] Tightening ScanSpec primary bounds when range predicate exists

2016-12-09 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Tightening ScanSpec primary bounds when range predicate exists .. Patch Set 2: (7 comments)

[kudu-CR](branch-1.2.x) KUDU-1776: Fix "kudu remote replica copy" connecting to wildcard address

2016-12-09 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1776: Fix "kudu remote_replica copy" connecting to wildcard address .. KUDU-1776: Fix "kudu remote_replica copy" connecting to wildcard

[kudu-CR](branch-1.2.x) KUDU-1776: Fix "kudu remote replica copy" connecting to wildcard address

2016-12-09 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1776: Fix "kudu remote_replica copy" connecting to wildcard address .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.2.x) KUDU-1776: Fix "kudu remote replica copy" connecting to wildcard address

2016-12-09 Thread Mike Percy (Code Review)
Mike Percy has uploaded a new change for review. http://gerrit.cloudera.org:8080/5442 Change subject: KUDU-1776: Fix "kudu remote_replica copy" connecting to wildcard address .. KUDU-1776: Fix "kudu remote_replica copy"

[kudu-CR] KUDU-1776: Fix "kudu remote replica copy" connecting to wildcard address

2016-12-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1776: Fix "kudu remote_replica copy" connecting to wildcard address .. Patch Set 8: Let's pull this in for 1.2.0 -- To view, visit

[kudu-CR] KUDU-1776: Fix "kudu remote replica copy" connecting to wildcard address

2016-12-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1776: Fix "kudu remote_replica copy" connecting to wildcard address .. Patch Set 8: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5378 To

[kudu-CR] KUDU-1776: Fix "kudu remote replica copy" connecting to wildcard address

2016-12-09 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: KUDU-1776: Fix "kudu remote_replica copy" connecting to wildcard address .. KUDU-1776: Fix "kudu remote_replica copy" connecting to wildcard address

[kudu-CR] KUDU-1776: Fix "kudu remote replica copy" connecting to wildcard address

2016-12-09 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: KUDU-1776: Fix "kudu remote_replica copy" connecting to wildcard address .. Patch Set 6: (9 comments)

[kudu-CR] KUDU-1643 Prune hash partitions based on IN-list predicates

2016-12-09 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1643 Prune hash partitions based on IN-list predicates .. Patch Set 8: (8 comments)

[kudu-CR] KUDU-1776: Fix "kudu remote replica copy" connecting to wildcard address

2016-12-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1776: Fix "kudu remote_replica copy" connecting to wildcard address .. Patch Set 6: (1 comment)

Re: [kudu-CR] KUDU-1386 NaN float and double values are not handled correctly

2016-12-09 Thread Dan Burkert
Sounds good. Since this was a somewhat controversial change, maybe you could send an email to the devlist laying out the current issue and possible solutions? Having a clear direction will help not get bogged down in the code review again. Thanks! - Dan On Fri, Dec 9, 2016 at 9:42 AM, Will

[kudu-CR] Make TabletCopyITest subclass ExternalMiniClusterITestBase

2016-12-09 Thread Mike Percy (Code Review)
Hello Dinesh Bhat, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5440 to review the following change. Change subject: Make TabletCopyITest subclass ExternalMiniClusterITestBase

[kudu-CR] KUDU-1798: [env] File manager broken on OS X

2016-12-09 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1798: [env] File manager broken on OS X .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/5438/2/src/kudu/util/env-test.cc File

[kudu-CR] KUDU-1776: Fix "kudu remote replica copy" connecting to wildcard address

2016-12-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1776: Fix "kudu remote_replica copy" connecting to wildcard address .. Patch Set 6: (8 comments)

[kudu-CR] KUDU-1608: Catalog manager can stop retrying DeleteTablet upon fatal errors

2016-12-09 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: KUDU-1608: Catalog manager can stop retrying DeleteTablet upon fatal errors .. Patch Set 3: > (1 comment) > > Looks good. As we discussed, please try to build

[kudu-CR] [spark] KUDU-1641 Pushdown SparkSQL In predicates

2016-12-09 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [spark] KUDU-1641 Pushdown SparkSQL In predicates .. Patch Set 2: Until we implement a max in the Kudu server I don't think it makes sense to limit the spark

[kudu-CR] KUDU-1776: Fix "kudu remote replica copy" connecting to wildcard address

2016-12-09 Thread Dinesh Bhat (Code Review)
Hello Mike Percy, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5378 to look at the new patch set (#6). Change subject: KUDU-1776: Fix "kudu remote_replica copy" connecting to wildcard address

[kudu-CR] [spark] KUDU-1641 Pushdown SparkSQL In predicates

2016-12-09 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: [spark] KUDU-1641 Pushdown SparkSQL In predicates .. Patch Set 2: > (1 comment) There are no limits client- or server-side in Kudu, or in Spark. What would be

[kudu-CR] KUDU-1386 NaN float and double values are not handled correctly

2016-12-09 Thread Will Berkeley (Code Review)
Will Berkeley has abandoned this change. Change subject: KUDU-1386 NaN float and double values are not handled correctly .. Abandoned Since this is so old, I'm just going to re-roll this as a new patch (soon™). -- To view,

[kudu-CR] KUDU-1595 and KUDU-1642: IS NOT NULL and IS NULL predicates

2016-12-09 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded a new change for review. http://gerrit.cloudera.org:8080/5439 Change subject: KUDU-1595 and KUDU-1642: IS NOT NULL and IS NULL predicates .. KUDU-1595 and KUDU-1642: IS NOT NULL and IS NULL

[kudu-CR] KUDU-1776: Fix "kudu remote replica copy" connecting to wildcard address

2016-12-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1776: Fix "kudu remote_replica copy" connecting to wildcard address .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/5378/5//COMMIT_MSG

[kudu-CR] Make delete table-test less flaky

2016-12-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Make delete_table-test less flaky .. Patch Set 1: (1 comment) > (1 comment) > > how about >

[kudu-CR] Make delete table-test less flaky

2016-12-09 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5421 to look at the new patch set (#2). Change subject: Make delete_table-test less flaky .. Make delete_table-test

[kudu-CR] Make delete table-test less flaky

2016-12-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Make delete_table-test less flaky .. Patch Set 1: > > how about > > TabletCopyClientSessionITest.TestStartTabletCopyWhileSourceBootstrapping Oops, forget what I

[kudu-CR] Change version in master to 1.3.0-SNAPSHOT

2016-12-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Change version in master to 1.3.0-SNAPSHOT .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5437 To unsubscribe, visit

[kudu-CR] Change version in master to 1.3.0-SNAPSHOT

2016-12-09 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: Change version in master to 1.3.0-SNAPSHOT .. Change version in master to 1.3.0-SNAPSHOT Change-Id: Ie52c36910dc0c333c1b3e86caa08edef2f45ad5e Reviewed-on:

[kudu-CR] KUDU-1798: [env] File manager broken on OS X

2016-12-09 Thread Dinesh Bhat (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5438 to look at the new patch set (#2). Change subject: KUDU-1798: [env] File manager broken on OS X ..

[kudu-CR] [env] File manager broken on OS X due to getrlimit returning RLIM INFINITY

2016-12-09 Thread Dinesh Bhat (Code Review)
Hello Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5438 to review the following change. Change subject: [env] File manager broken on OS X due to getrlimit returning RLIM_INFINITY