[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-11 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8951 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. KUDU-2251: rowset size can overflow int in RowSetInfo

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-11 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8951 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. Patch Set 7: I've backed out the integration test, since it

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-11 Thread Dan Burkert (Code Review)
Hello Tidy Bot, David Ribeiro Alves, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8951 to look at the new patch set (#7). Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8951 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. Patch Set 6: Without profiling it I don't have a great

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8951 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. Patch Set 6: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-08 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8951 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. Patch Set 6: I changed the asserts and expects to checks, and

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8951 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. Patch Set 6: (1 comment)

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-08 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/8951 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. Patch Set 6: (2 comments)

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-08 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8951 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. Patch Set 6: OK I think I've satisfied everyone's requests WRT

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-05 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8951 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. Patch Set 5: (2 comments)

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-05 Thread Dan Burkert (Code Review)
Hello Tidy Bot, David Ribeiro Alves, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8951 to look at the new patch set (#6). Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8951 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. Patch Set 5: (2 comments)

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-05 Thread Dan Burkert (Code Review)
Hello Tidy Bot, David Ribeiro Alves, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8951 to look at the new patch set (#5). Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-05 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8951 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. Patch Set 5: (4 comments)

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-05 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8951 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. Patch Set 4: (4 comments)

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/8951 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-05 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8951 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. Patch Set 1: (3 comments)

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-05 Thread Dan Burkert (Code Review)
Hello Tidy Bot, David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8951 to look at the new patch set (#4). Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-05 Thread Dan Burkert (Code Review)
Hello Tidy Bot, David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8951 to look at the new patch set (#3). Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/8951 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. Patch Set 1: (4 comments)

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-05 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8951 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-05 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8951 Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. KUDU-2251: rowset size can overflow int in RowSetInfo This