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

Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8830 ) Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client) .. KUDU-721: Support for Decimal type, Part 1

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

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 32: Code-Review+2 Thanks everyone for the

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

Alexey Serbin 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 32: Code-Review+2 (3 comments)

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#32). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#30). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

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 28: (8 comments)

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#29). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

Alexey Serbin 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 28: (8 comments) LGTM, just a few nits.

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

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 28: Code-Review+2 (1 comment)

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

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 28: (1 comment)

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

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 28: (1 comment)

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#28). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#27). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#26). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#25). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#24). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#22). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

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: (5 comments)

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#21). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

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)

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

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)

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

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)

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#20). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#19). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

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 17: (4 comments)

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#18). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

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: (8 comments)

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#17). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

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 16: (21 comments)

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#16). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

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: (18 comments)

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#15). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#14). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#13). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#12). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

Alexey Serbin 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 11: (5 comments)

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#11). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#10). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#9). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

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 8: (21 comments) Still tweaking the

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

Todd Lipcon 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 8: (23 comments) I think this could do

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#8). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

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 1: (25 comments) I need to tweak the

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

Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#7). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

Alexey Serbin 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 6: (12 comments) It seems I took a quick

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

Adar Dembo 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 6: (4 comments) I only looked at the public

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

Hello Tidy Bot, Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#6). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#5). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#4). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#3). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

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

Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8830 to look at the new patch set (#2). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)