[kudu-CR] Implement BloomFilter Predicate in server side.

2018-10-04 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11100 ) Change subject: Implement BloomFilter Predicate in server side. .. Patch Set 9: (15 comments)

[kudu-CR] KUDU-2563: [spark] Use the scanner keepAlive API

2018-10-04 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11571 ) Change subject: KUDU-2563: [spark] Use the scanner keepAlive API .. Patch Set 4: (1 comment)

[kudu-CR] KUDU-2563: [spark] Use the scanner keepAlive API

2018-10-04 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11571 ) Change subject: KUDU-2563: [spark] Use the scanner keepAlive API .. KUDU-2563: [spark] Use the scanner keepAlive API Adds

[kudu-CR] KUDU-2563: [spark] Use the scanner keepAlive API

2018-10-04 Thread Grant Henke (Code Review)
Hello Mike Percy, Dan Burkert, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11571 to look at the new patch set (#4). Change subject: KUDU-2563: [spark] Use the scanner keepAlive API

[kudu-CR] KUDU-2563: [spark] Use the scanner keepAlive API

2018-10-04 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11571 ) Change subject: KUDU-2563: [spark] Use the scanner keepAlive API .. Patch Set 4: Verified+1 Code-Review+2 Carrying the +2 forward

[kudu-CR] KUDU-2563: [spark] Use the scanner keepAlive API

2018-10-04 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11571 ) Change subject: KUDU-2563: [spark] Use the scanner keepAlive API .. Patch Set 3: Code-Review+2 (1 comment)

[kudu-CR] [tools] KUDU-2179: Have ksck not use a single snapshot for all tablets

2018-10-04 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Alexey Serbin, Attila Bukor, Kudu Jenkins, Andrew Wong, Adar Dembo, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11554 to look at the new patch set (#11). Change subject: [tools] KUDU-2179: Have ksck not use a

[kudu-CR] KUDU-2563: [spark] Use the scanner keepAlive API

2018-10-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11571 ) Change subject: KUDU-2563: [spark] Use the scanner keepAlive API .. Patch Set 3: Code-Review+2 Would be nice to get another review

[kudu-CR] [tools] KUDU-2179: Have ksck not use a single snapshot for all tablets

2018-10-04 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11554 ) Change subject: [tools] KUDU-2179: Have ksck not use a single snapshot for all tablets .. Patch Set 9: (6 comments)

[kudu-CR] KUDU-2563: [spark] Use the scanner keepAlive API

2018-10-04 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11571 ) Change subject: KUDU-2563: [spark] Use the scanner keepAlive API .. Patch Set 2: (7 comments) > It would be good to double check that

[kudu-CR] KUDU-2563: [spark] Use the scanner keepAlive API

2018-10-04 Thread Grant Henke (Code Review)
Hello Mike Percy, Dan Burkert, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11571 to look at the new patch set (#3). Change subject: KUDU-2563: [spark] Use the scanner keepAlive API

[kudu-CR] docs: add a note about RAID-0

2018-10-04 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11580 ) Change subject: docs: add a note about RAID-0 .. docs: add a note about RAID-0 Adds a note that a single RAID-0 device with

[kudu-CR] docs: add a note about RAID-0

2018-10-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11580 ) Change subject: docs: add a note about RAID-0 .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/11580 To

[kudu-CR] docs: add a note about RAID-0

2018-10-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11580 ) Change subject: docs: add a note about RAID-0 .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-2563: [spark] Use the scanner keepAlive API

2018-10-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11571 ) Change subject: KUDU-2563: [spark] Use the scanner keepAlive API .. Patch Set 2: (3 comments) It would be good to double check that

[kudu-CR] docs: add a note about RAID-0

2018-10-04 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11580 ) Change subject: docs: add a note about RAID-0 .. Patch Set 3: (1 comment)

[kudu-CR] docs: add a note about RAID-0

2018-10-04 Thread Andrew Wong (Code Review)
Hello Alex Rodoni, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11580 to look at the new patch set (#3). Change subject: docs: add a note about RAID-0 .. docs:

[kudu-CR] [tools] KUDU-2179: Have ksck not use a single snapshot for all tablets

2018-10-04 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11554 ) Change subject: [tools] KUDU-2179: Have ksck not use a single snapshot for all tablets .. Patch Set 10: (5 comments)

[kudu-CR] docs: update docs for CFile checksum handling

2018-10-04 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11581 ) Change subject: docs: update docs for CFile checksum handling .. docs: update docs for CFile checksum handling Notes that the

[kudu-CR] docs: add a note about RAID-0

2018-10-04 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11580 ) Change subject: docs: add a note about RAID-0 .. Patch Set 2: Verified+1 Overriding -1 since this is a docs patch -- To view, visit

[kudu-CR] [master] extra tests for the placement policy

2018-10-04 Thread Alexey Serbin (Code Review)
Hello Will Berkeley, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11562 to look at the new patch set (#3). Change subject: [master] extra tests for the placement policy ..

[kudu-CR] docs: add a note about RAID-0

2018-10-04 Thread Andrew Wong (Code Review)
Andrew Wong has removed a vote on this change. Change subject: docs: add a note about RAID-0 .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/11580 To unsubscribe, visit

[kudu-CR] [master] extra tests for the placement policy

2018-10-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11562 ) Change subject: [master] extra tests for the placement policy .. Patch Set 1: (7 comments)

[kudu-CR] [tools] KUDU-2179: Have ksck not use a single snapshot for all tablets

2018-10-04 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11554 ) Change subject: [tools] KUDU-2179: Have ksck not use a single snapshot for all tablets .. Patch Set 7: (18 comments)

[kudu-CR] KUDU-2563: [spark] Use the scanner keepAlive API

2018-10-04 Thread Grant Henke (Code Review)
Hello Mike Percy, Dan Burkert, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11571 to look at the new patch set (#2). Change subject: KUDU-2563: [spark] Use the scanner keepAlive API

[kudu-CR] docs: add a note about RAID-0

2018-10-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11580 ) Change subject: docs: add a note about RAID-0 .. Patch Set 1: (1 comment)

[kudu-CR] docs: update docs for CFile checksum handling

2018-10-04 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11581 ) Change subject: docs: update docs for CFile checksum handling .. Patch Set 1: Verified+1 Code-Review+2 -- To view, visit

[kudu-CR] [tools] KUDU-2179: Have ksck not use a single snapshot for all tablets

2018-10-04 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Alexey Serbin, Attila Bukor, Kudu Jenkins, Andrew Wong, Adar Dembo, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11554 to look at the new patch set (#10). Change subject: [tools] KUDU-2179: Have ksck not use a

[kudu-CR] docs: update docs for CFile checksum handling

2018-10-04 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11581 Change subject: docs: update docs for CFile checksum handling .. docs: update docs for CFile checksum handling Notes that the behavior

[kudu-CR] docs: add a note about RAID-0

2018-10-04 Thread Andrew Wong (Code Review)
Hello Alex Rodoni, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11580 to look at the new patch set (#2). Change subject: docs: add a note about RAID-0 .. docs:

[kudu-CR] docs: add a note about RAID-0

2018-10-04 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11580 ) Change subject: docs: add a note about RAID-0 .. Patch Set 2: (3 comments)

[kudu-CR] docs: add a note about RAID-0

2018-10-04 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/11580 ) Change subject: docs: add a note about RAID-0 .. Patch Set 1: (2 comments)

[kudu-CR] docs: add a note about RAID-0

2018-10-04 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11580 Change subject: docs: add a note about RAID-0 .. docs: add a note about RAID-0 Adds a note that a single RAID-0 device with multiple

[kudu-CR] [master] extra tests for the placement policy

2018-10-04 Thread Alexey Serbin (Code Review)
Hello Will Berkeley, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11562 to look at the new patch set (#2). Change subject: [master] extra tests for the placement policy .. [master]

[kudu-CR] Remove Pandas as a Python Client Requirement

2018-10-04 Thread Jordan Birdsell (Code Review)
Jordan Birdsell has posted comments on this change. ( http://gerrit.cloudera.org:8080/11557 ) Change subject: Remove Pandas as a Python Client Requirement .. Patch Set 5: Code-Review+1 I'm good with this change but would like

[kudu-CR] Remove Pandas as a Python Client Requirement

2018-10-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11557 ) Change subject: Remove Pandas as a Python Client Requirement .. Patch Set 5: Code-Review+2 (1 comment)

[kudu-CR] Remove Pandas as a Python Client Requirement

2018-10-04 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11557 ) Change subject: Remove Pandas as a Python Client Requirement .. Remove Pandas as a Python Client Requirement This is to remove

[kudu-CR] KUDU-1592 Make warnings about FBM more ominous

2018-10-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11579 ) Change subject: KUDU-1592 Make warnings about FBM more ominous .. Patch Set 1: (3 comments)

[kudu-CR] [tools] KUDU-2179: Have ksck not use a single snapshot for all tablets

2018-10-04 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11554 ) Change subject: [tools] KUDU-2179: Have ksck not use a single snapshot for all tablets .. Patch Set 9: (1 comment)

[kudu-CR] [master] extra tests for the placement policy

2018-10-04 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11562 ) Change subject: [master] extra tests for the placement policy .. Patch Set 1: (7 comments)

[kudu-CR] [tools] KUDU-2179: Have ksck not use a single snapshot for all tablets

2018-10-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11554 ) Change subject: [tools] KUDU-2179: Have ksck not use a single snapshot for all tablets .. Patch Set 9: (18 comments) Skimmed trough

[kudu-CR] Remove Pandas as a Python Client Requirement

2018-10-04 Thread Anonymous Coward (Code Review)
a...@phdata.io has posted comments on this change. ( http://gerrit.cloudera.org:8080/11557 ) Change subject: Remove Pandas as a Python Client Requirement .. Patch Set 5: (1 comment)

[kudu-CR] Remove Pandas as a Python Client Requirement

2018-10-04 Thread Anonymous Coward (Code Review)
Hello Kudu Jenkins, Jordan Birdsell, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11557 to look at the new patch set (#5). Change subject: Remove Pandas as a Python Client Requirement

[kudu-CR] Remove Pandas as a Python Client Requirement

2018-10-04 Thread Anonymous Coward (Code Review)
a...@phdata.io has posted comments on this change. ( http://gerrit.cloudera.org:8080/11557 ) Change subject: Remove Pandas as a Python Client Requirement .. Patch Set 4: (6 comments)

[kudu-CR] Remove Pandas as a Python Client Requirement

2018-10-04 Thread Anonymous Coward (Code Review)
Hello Kudu Jenkins, Jordan Birdsell, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11557 to look at the new patch set (#4). Change subject: Remove Pandas as a Python Client Requirement

[kudu-CR] [tools] KUDU-2179: Have ksck not use a single snapshot for all tablets

2018-10-04 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Alexey Serbin, Attila Bukor, Kudu Jenkins, Andrew Wong, Adar Dembo, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11554 to look at the new patch set (#9). Change subject: [tools] KUDU-2179: Have ksck not use a

[kudu-CR] KUDU-1592 Make warnings about FBM more ominous

2018-10-04 Thread Attila Bukor (Code Review)
Hello Jean-Daniel Cryans, Adar Dembo, Grant Henke, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/11579 to review the following change. Change subject: KUDU-1592 Make warnings about FBM more ominous

[kudu-CR] [tools] KUDU-2179: Have ksck not use a single snapshot for all tablets

2018-10-04 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11554 ) Change subject: [tools] KUDU-2179: Have ksck not use a single snapshot for all tablets .. Patch Set 7: (18 comments)

[kudu-CR] [tools] KUDU-2179: Have ksck not use a single snapshot for all tablets

2018-10-04 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Alexey Serbin, Attila Bukor, Kudu Jenkins, Andrew Wong, Adar Dembo, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11554 to look at the new patch set (#8). Change subject: [tools] KUDU-2179: Have ksck not use a