[kudu-CR] [tools] ksck checksums: Add KsckChecksummer class

2018-10-02 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11498 ) Change subject: [tools] ksck checksums: Add KsckChecksummer class .. Patch Set 8: (3 comments)

[kudu-CR] [tools] ksck checksums: Add KsckChecksummer class

2018-10-02 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Alexey Serbin, Attila Bukor, Kudu Jenkins, Andrew Wong, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11498 to look at the new patch set (#9). Change subject: [tools] ksck checksums: Add KsckChecksummer class

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

2018-10-02 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11554 to look at the new patch set (#7). Change subject: [tools] KUDU-2179: Have ksck not use a single snapshot for all tablets

[kudu-CR] Extract connection retrying and HA support from HmsCatalog

2018-10-02 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11570 ) Change subject: Extract connection retrying and HA support from HmsCatalog .. Patch Set 2: (5 comments)

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

2018-10-02 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( 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 scheduled

[kudu-CR] Extract connection retrying and HA support from HmsCatalog

2018-10-02 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11570 ) Change subject: Extract connection retrying and HA support from HmsCatalog .. Patch Set 2: (1 comment)

[kudu-CR] Extract connection retrying and HA support from HmsCatalog

2018-10-02 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11570 ) Change subject: Extract connection retrying and HA support from HmsCatalog .. Patch Set 2: (3 comments)

[kudu-CR] [tools] ksck checksums: Add KsckChecksummer class

2018-10-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11498 ) Change subject: [tools] ksck checksums: Add KsckChecksummer class .. Patch Set 8: Code-Review+2 (3 comments)

[kudu-CR] [hms] enable 'notifications.add.thrift.objects' in mini HMS

2018-10-02 Thread Hao Hao (Code Review)
Hello Dan Burkert, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11566 to look at the new patch set (#3). Change subject: [hms] enable 'notifications.add.thrift.objects' in mini HMS

[kudu-CR] [hms] enable 'notifications.add.thrift.objects' in mini HMS

2018-10-02 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11566 ) Change subject: [hms] enable 'notifications.add.thrift.objects' in mini HMS .. Patch Set 2: (1 comment)

[kudu-CR] [hms] enable 'notifications.add.thrift.objects' in mini HMS

2018-10-02 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11566 ) Change subject: [hms] enable 'notifications.add.thrift.objects' in mini HMS .. Patch Set 2: Code-Review+1 (1 comment)

[kudu-CR] [hms] enable 'notifications.add.thrift.objects' in mini HMS

2018-10-02 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11566 ) Change subject: [hms] enable 'notifications.add.thrift.objects' in mini HMS .. Patch Set 2: (1 comment)

[kudu-CR] [hms] enable 'notifications.add.thrift.objects' in mini HMS

2018-10-02 Thread Hao Hao (Code Review)
Hello Dan Burkert, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11566 to look at the new patch set (#2). Change subject: [hms] enable 'notifications.add.thrift.objects' in mini HMS

[kudu-CR] Extract connection retrying and HA support from HmsCatalog

2018-10-02 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11570 ) Change subject: Extract connection retrying and HA support from HmsCatalog .. Patch Set 2: (2 comments)

[kudu-CR] Extract connection retrying and HA support from HmsCatalog

2018-10-02 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Andrew Wong, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11570 to look at the new patch set (#2). Change subject: Extract connection retrying and HA support from HmsCatalog

[kudu-CR] Extract connection retrying and HA support from HmsCatalog

2018-10-02 Thread Dan Burkert (Code Review)
Hello Andrew Wong, Adar Dembo, Hao Hao, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/11570 to review the following change. Change subject: Extract connection retrying and HA support from HmsCatalog

[kudu-CR] [tools] ksck checksums: Add KsckChecksummer class

2018-10-02 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Alexey Serbin, Attila Bukor, Kudu Jenkins, Andrew Wong, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11498 to look at the new patch set (#8). Change subject: [tools] ksck checksums: Add KsckChecksummer class

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

2018-10-02 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11554 to look at the new patch set (#6). Change subject: [tools] KUDU-2179: Have ksck not use a single snapshot for all tablets

[kudu-CR] [hms] enable 'notifications.add.thrift.objects' in mini HMS

2018-10-02 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11566 ) Change subject: [hms] enable 'notifications.add.thrift.objects' in mini HMS .. Patch Set 1: (1 comment)

[kudu-CR] [tools] ksck checksums: Add KsckChecksummer class

2018-10-02 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11498 ) Change subject: [tools] ksck checksums: Add KsckChecksummer class .. Patch Set 6: (6 comments)

[kudu-CR] [hms] enable 'notifications.add.thrift.objects' in mini HMS

2018-10-02 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11566 Change subject: [hms] enable 'notifications.add.thrift.objects' in mini HMS .. [hms] enable 'notifications.add.thrift.objects' in mini HMS

[kudu-CR] [tools] ksck checksums: Add KsckChecksummer class

2018-10-02 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Alexey Serbin, Attila Bukor, Kudu Jenkins, Andrew Wong, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11498 to look at the new patch set (#7). Change subject: [tools] ksck checksums: Add KsckChecksummer class

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

2018-10-02 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11554 to look at the new patch set (#5). Change subject: [tools] KUDU-2179: Have ksck not use a single snapshot for all tablets

[kudu-CR] [tools] ksck checksums: Add KsckChecksummer class

2018-10-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11498 ) Change subject: [tools] ksck checksums: Add KsckChecksummer class .. Patch Set 6: (1 comment)

[kudu-CR] [tools] ksck checksums: Add KsckChecksummer class

2018-10-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11498 ) Change subject: [tools] ksck checksums: Add KsckChecksummer class .. Patch Set 6: (4 comments)

[kudu-CR] [test] Migrate the Spark tests to use the new KuduRule

2018-10-02 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11548 ) Change subject: [test] Migrate the Spark tests to use the new KuduRule .. [test] Migrate the Spark tests to use the new KuduRule

[kudu-CR] [Java] Upgrade Dependencies

2018-10-02 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11541 ) Change subject: [Java] Upgrade Dependencies .. [Java] Upgrade Dependencies Upgrades the Java dependencies and Gradle versions.

[kudu-CR] [spark] Add KuduReadOptions to encapsulate the read parameters

2018-10-02 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11537 ) Change subject: [spark] Add KuduReadOptions to encapsulate the read parameters .. [spark] Add KuduReadOptions to encapsulate the

[kudu-CR] [test] Move BaseKuduTest to a Junit Rule

2018-10-02 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11547 ) Change subject: [test] Move BaseKuduTest to a Junit Rule .. [test] Move BaseKuduTest to a Junit Rule This patch moves all of

[kudu-CR] HMS integration: provide Java API to override owner during table create

2018-10-02 Thread Andrew Wong (Code Review)
Andrew Wong has removed Andrew Wong from this change. ( http://gerrit.cloudera.org:8080/11413 ) Change subject: HMS integration: provide Java API to override owner during table create .. Removed reviewer Andrew Wong. -- To

[kudu-CR] tserver: log a message when a scanner is not found

2018-10-02 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11516 ) Change subject: tserver: log a message when a scanner is not found .. tserver: log a message when a scanner is not found Right

[kudu-CR] [test] Migrate the Spark tests to use the new KuduRule

2018-10-02 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11548 ) Change subject: [test] Migrate the Spark tests to use the new KuduRule .. Patch Set 6: Code-Review+2 -- To view, visit

[kudu-CR] [test] Move BaseKuduTest to a Junit Rule

2018-10-02 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11547 ) Change subject: [test] Move BaseKuduTest to a Junit Rule .. Patch Set 7: Code-Review+2 -- To view, visit

[kudu-CR] [test] Move BaseKuduTest to a Junit Rule

2018-10-02 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11547 ) Change subject: [test] Move BaseKuduTest to a Junit Rule .. Patch Set 1: (1 comment)

[kudu-CR] [test] Move BaseKuduTest to a Junit Rule

2018-10-02 Thread Grant Henke (Code Review)
Hello Mike Percy, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11547 to look at the new patch set (#7). Change subject: [test] Move BaseKuduTest to a Junit Rule

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 7: (1 comment)

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-02 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 7: (1 comment)

[kudu-CR] [tools] ksck checksums: Add KsckChecksummer class

2018-10-02 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11498 ) Change subject: [tools] ksck checksums: Add KsckChecksummer class .. Patch Set 6: Code-Review+1 (1 comment)

[kudu-CR] [tools] ksck checksums: Factor out of main ksck code

2018-10-02 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11488 ) Change subject: [tools] ksck checksums: Factor out of main ksck code .. [tools] ksck checksums: Factor out of main ksck code

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 7: (1 comment)

[kudu-CR] [tools] ksck checksums: Add KsckChecksummer class

2018-10-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11498 ) Change subject: [tools] ksck checksums: Add KsckChecksummer class .. Patch Set 6: Code-Review+1 (1 comment) I'm planning to take

[kudu-CR] [tools] ksck checksums: Factor out of main ksck code

2018-10-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11488 ) Change subject: [tools] ksck checksums: Factor out of main ksck code .. Patch Set 9: Code-Review+2 (3 comments)

[kudu-CR] [tools] ksck checksums: Factor out of main ksck code

2018-10-02 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Alexey Serbin, Attila Bukor, Kudu Jenkins, Andrew Wong, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11488 to look at the new patch set (#9). Change subject: [tools] ksck checksums: Factor out of main ksck code

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

2018-10-02 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11554 to look at the new patch set (#4). Change subject: [tools] KUDU-2179: Have ksck not use a single snapshot for all tablets

[kudu-CR] [tools] ksck checksums: Add KsckChecksummer class

2018-10-02 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11498 ) Change subject: [tools] ksck checksums: Add KsckChecksummer class .. Patch Set 6: (1 comment)

[kudu-CR] [tools] ksck checksums: Factor out of main ksck code

2018-10-02 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11488 ) Change subject: [tools] ksck checksums: Factor out of main ksck code .. Patch Set 8: (8 comments)

[kudu-CR] [tools] ksck checksums: Add KsckChecksummer class

2018-10-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11498 ) Change subject: [tools] ksck checksums: Add KsckChecksummer class .. Patch Set 5: (3 comments)