Jim Apple has posted comments on this change. ( http://gerrit.cloudera.org:8080/11810 )
Change subject: IMPALA-5031: Make UBSAN-friendly arithmetic generic ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/11810/1/be/src/util/arithmetic-util.h File be/src/util/arithmetic-util.h: http://gerrit.cloudera.org:8080/#/c/11810/1/be/src/util/arithmetic-util.h@116 PS1, Line 116: enum class Ring { INTEGER, FLOAT, NEITHER }; > I feel like there's some context missing here. I guess it's to enable gener I added some more comments. I'll wait for you to have a chance to ask clarifying questions before I kickoff the pre-merge job. -- To view, visit http://gerrit.cloudera.org:8080/11810 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I73bec71e59c5a921003d0ebca52a1d4e49bbef66 Gerrit-Change-Number: 11810 Gerrit-PatchSet: 1 Gerrit-Owner: Jim Apple <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jim Apple <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Comment-Date: Sat, 03 Nov 2018 18:19:31 +0000 Gerrit-HasComments: Yes
