Dan Hecht has posted comments on this change. Change subject: IMPALA-4809: Codegen support for DECIMAL_V2 ......................................................................
Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5950/1/be/src/exec/partitioned-aggregation-node.cc File be/src/exec/partitioned-aggregation-node.cc: PS1, Line 1740: output_type > Tim fixed this as well in https://gerrit.cloudera.org/#/c/5161/23. Same fix or, after seeing the full patch, i think you can just revert this to avoid the conflict. -- To view, visit http://gerrit.cloudera.org:8080/5950 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2434d240f65b81389b8a8ba027f980a0e1d1f981 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Michael Ho <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-HasComments: Yes
