[ https://issues.apache.org/jira/browse/IMPALA-5031?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16765660#comment-16765660 ]
ASF subversion and git services commented on IMPALA-5031: --------------------------------------------------------- Commit 6fc05a6f7385bf21de1c01a3071b9a91efc85b07 in impala's branch refs/heads/master from Jim Apple [ https://gitbox.apache.org/repos/asf?p=impala.git;h=6fc05a6 ] IMPALA-5031: signed overflow is undefined behavior Tis patch fixes a signed overflow in the test StringToDecimal.LargeDecimals. The interesting part of the backtrace is: util/string-parser.h:397:14: runtime error: signed integer overflow: 0x4b3b4ca85a86c47a098a223fffffffff * 10 cannot be represented in type '__int128' #0 void StringParser::ApplyExponent<__int128>(int, int, signed char, __int128*, int*, int*) util/string-parser.h:397:14 #1 DecimalValue<__int128> StringParser::StringToDecimal<__int128> (char const*, int, int, int, bool, StringParser::ParseResult*) util/string-parser.h:221:5 #2 void VerifyParse<__int128>(string const&, int, int, bool, DecimalValue<__int128> const&, StringParser::ParseResult) runtime/decimal-test.cc:53:25 #3 void VerifyParse<__int128>(string const&, int, int, DecimalValue<__int128> const&, StringParser::ParseResult) runtime/decimal-test.cc:65:3 #4 StringToDecimal_LargeDecimals_Test::TestBody() runtime/decimal-test.cc:443:3 Change-Id: Ifb4effa291e1e4dac62b84251c74c483d99b06e7 Reviewed-on: http://gerrit.cloudera.org:8080/12426 Reviewed-by: Jim Apple <jbapple-imp...@apache.org> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> > UBSAN clean and method for testing UBSAN cleanliness > ---------------------------------------------------- > > Key: IMPALA-5031 > URL: https://issues.apache.org/jira/browse/IMPALA-5031 > Project: IMPALA > Issue Type: Task > Components: Backend, Infrastructure > Affects Versions: Impala 2.9.0 > Reporter: Jim Apple > Assignee: Jim Apple > Priority: Minor > > http://releases.llvm.org/3.8.0/tools/clang/docs/UndefinedBehaviorSanitizer.html > builds are supported after https://gerrit.cloudera.org/#/c/6186/, but > Impala's test suite triggers many errors under UBSAN. Those errors should be > fixed and then there should be a way to run the test suite under UBSAN and > fail if there were any errors detected. -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: issues-all-unsubscr...@impala.apache.org For additional commands, e-mail: issues-all-h...@impala.apache.org