[kudu-CR] [tests] Add more test coverage for INT128 columns

2018-02-05 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9193 ) Change subject: [tests] Add more test coverage for INT128 columns .. [tests] Add more test coverage for INT128 columns Improves

[kudu-CR] [tests] Add more test coverage for INT128 columns

2018-02-05 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9193 ) Change subject: [tests] Add more test coverage for INT128 columns .. Patch Set 9: Code-Review+2 -- To view, visit

[kudu-CR] [tests] Add more test coverage for INT128 columns

2018-02-05 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/9193 ) Change subject: [tests] Add more test coverage for INT128 columns .. Patch Set 9: (2 comments)

[kudu-CR] [tests] Add more test coverage for INT128 columns

2018-02-05 Thread Grant Henke (Code Review)
Hello Tidy Bot, Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9193 to look at the new patch set (#9). Change subject: [tests] Add more test coverage for INT128 columns

[kudu-CR] [tests] Add more test coverage for INT128 columns

2018-02-05 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9193 ) Change subject: [tests] Add more test coverage for INT128 columns .. Patch Set 8: (2 comments) LGTM, but seems like it may be a good

[kudu-CR] [tests] Add more test coverage for INT128 columns

2018-02-02 Thread Grant Henke (Code Review)
Hello Tidy Bot, Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9193 to look at the new patch set (#5). Change subject: [tests] Add more test coverage for INT128 columns

[kudu-CR] [tests] Add more test coverage for INT128 columns

2018-02-02 Thread Grant Henke (Code Review)
Hello Tidy Bot, Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9193 to look at the new patch set (#4). Change subject: [tests] Add more test coverage for INT128 columns

[kudu-CR] [tests] Add more test coverage for INT128 columns

2018-02-02 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/9193 ) Change subject: [tests] Add more test coverage for INT128 columns .. Patch Set 2: (2 comments)

[kudu-CR] [tests] Add more test coverage for INT128 columns

2018-02-02 Thread Grant Henke (Code Review)
Hello Tidy Bot, Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9193 to look at the new patch set (#3). Change subject: [tests] Add more test coverage for INT128 columns

[kudu-CR] [tests] Add more test coverage for INT128 columns

2018-02-02 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9193 ) Change subject: [tests] Add more test coverage for INT128 columns .. Patch Set 2: (2 comments)

[kudu-CR] [tests] Add more test coverage for INT128 columns

2018-02-02 Thread Grant Henke (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9193 to look at the new patch set (#2). Change subject: [tests] Add more test coverage for INT128 columns ..

[kudu-CR] [tests] Add more test coverage for INT128 columns

2018-02-02 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9193 Change subject: [tests] Add more test coverage for INT128 columns .. [tests] Add more test coverage for INT128 columns Improves test