[kudu-CR] [tablet] Add a pre-compaction check to avoid compactions in the past

2018-01-30 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8887 to look at the new patch set (#12). Change subject: [tablet] Add a pre-compaction check to avoid compactions in the past ..

[kudu-CR] [tests] de-flaking DontEvictIfRemainingConfigIsUnstable

2018-01-30 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9126 ) Change subject: [tests] de-flaking DontEvictIfRemainingConfigIsUnstable .. Patch Set 2: Code-Review+2 Thanks for trying to tune it down.

[kudu-CR] [tests] de-flaking DontEvictIfRemainingConfigIsUnstable

2018-01-30 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9126 ) Change subject: [tests] de-flaking DontEvictIfRemainingConfigIsUnstable .. [tests] de-flaking DontEvictIfRemainingConfigIsUnstable

[kudu-CR] Don't rely on pending config OpId index for peer promotion

2018-01-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9161 ) Change subject: Don't rely on pending config OpId index for peer promotion .. Patch Set 3: (1 comment) http://gerrit.cloudera.or

[kudu-CR] Simplify OpId/Timestamp assignment and make it atomic

2018-01-30 Thread David Ribeiro Alves (Code Review)
Hello Tidy Bot, Mike Percy, Alexey Serbin, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7221 to look at the new patch set (#16). Change subject: Simplify OpId/Timestamp assignment and make it atomic .

[kudu-CR] KUDU-2191 (4/n): HMS Thrift client fault handling

2018-01-30 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8494 ) Change subject: KUDU-2191 (4/n): HMS Thrift client fault handling .. Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/8494/8

[kudu-CR] rpc: micro-optimize delayed task handling

2018-01-30 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9048 ) Change subject: rpc: micro-optimize delayed task handling .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/9048/3/src/kud

[kudu-CR] [thirdparty] fix sized-deallocation issue on OS X 10.11

2018-01-30 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9159 ) Change subject: [thirdparty] fix sized-deallocation issue on OS X 10.11 .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c

[kudu-CR] Don't rely on pending config OpId index for peer promotion

2018-01-30 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9161 ) Change subject: Don't rely on pending config OpId index for peer promotion .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#

[kudu-CR] [tablet] Add a pre-compaction check to avoid compactions in the past

2018-01-30 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8887 to look at the new patch set (#10). Change subject: [tablet] Add a pre-compaction check to avoid compactions in the past ..

[kudu-CR] Simplify OpId/Timestamp assignment and make it atomic

2018-01-30 Thread David Ribeiro Alves (Code Review)
Hello Tidy Bot, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7221 to look at the new patch set (#15). Change subject: Simplify OpId/Timestamp assignment and make it atomic

[kudu-CR] WIP: Simplify OpId/Timestamp assignment and make it atomic

2018-01-30 Thread David Ribeiro Alves (Code Review)
Hello Tidy Bot, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7221 to look at the new patch set (#14). Change subject: WIP: Simplify OpId/Timestamp assignment and make it atomic ...

[kudu-CR] KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

2018-01-30 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8830 ) Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client) .. Patch Set 20: (4 comments) http://gerrit.cloudera.org

[kudu-CR] KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

2018-01-30 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/8830 ) Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client) .. Patch Set 14: (1 comment) http://gerrit.cloudera.org:

[kudu-CR] KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

2018-01-30 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8830 ) Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client) .. Patch Set 14: (2 comments) http://gerrit.cloudera.org

[kudu-CR] [spark] enable scan locality by default

2018-01-30 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9163 Change subject: [spark] enable scan locality by default .. [spark] enable scan locality by default Commit 3abca98c5 introduced support to take

[kudu-CR] docs: fix number of number masters

2018-01-30 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/9091 ) Change subject: docs: fix number of number masters .. Patch Set 5: (1 comment) > Patch Set 4: > > (1 comment) http://gerrit.cloudera.o

[kudu-CR] Don't rely on pending config OpId index for peer promotion

2018-01-30 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9161 to look at the new patch set (#3). Change subject: Don't rely on pending config OpId index for peer promotion .. Don

[kudu-CR] docs: fix number of number masters

2018-01-30 Thread Attila Bukor (Code Review)
Hello Alex Rodoni, Kudu Jenkins, Andrew Wong, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9091 to look at the new patch set (#5). Change subject: docs: fix number of number masters

[kudu-CR] Don't rely on pending config OpId index for peer promotion

2018-01-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9161 Change subject: Don't rely on pending config OpId index for peer promotion .. Don't rely on pending config OpId index for peer prom

[kudu-CR] Simplify OpId/Timestamp assignment and make it atomic

2018-01-30 Thread David Ribeiro Alves (Code Review)
Hello Tidy Bot, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7221 to look at the new patch set (#12). Change subject: Simplify OpId/Timestamp assignment and make it atomic

[kudu-CR] [thirdparty] fix sized-deallocation issue on OS X 10.11

2018-01-30 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9159 ) Change subject: [thirdparty] fix sized-deallocation issue on OS X 10.11 .. Patch Set 1: Verified+1 unrelated breakage in KuduMetastoreP

[kudu-CR] [thirdparty] fix sized-deallocation issue on OS X 10.11

2018-01-30 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/9159 ) Change subject: [thirdparty] fix sized-deallocation issue on OS X 10.11 .. Removed reviewer Kudu Jenkins with the following vote