Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16006 )
Change subject: Revert "IMPALA-9781: Fix GCC 7 unaligned 128-bit loads / stores" ...................................................................... Revert "IMPALA-9781: Fix GCC 7 unaligned 128-bit loads / stores" The change in decimal-util.h introduced undefined behavior. See IMPALA-9800. This reverts commit 227da84c3757eb857008e7b82aad622ed959eb84. Change-Id: Id2b2e43c478a220ff545fdbca712e47905c8d22b Reviewed-on: http://gerrit.cloudera.org:8080/16006 Reviewed-by: Joe McDonnell <joemcdonn...@cloudera.com> Reviewed-by: Tim Armstrong <tarmstr...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- M be/src/exprs/slot-ref.cc M be/src/util/decimal-util.h M be/src/util/dict-encoding.h 3 files changed, 4 insertions(+), 9 deletions(-) Approvals: Joe McDonnell: Looks good to me, approved Tim Armstrong: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/16006 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Id2b2e43c478a220ff545fdbca712e47905c8d22b Gerrit-Change-Number: 16006 Gerrit-PatchSet: 2 Gerrit-Owner: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>