[ https://issues.apache.org/jira/browse/HIVE-10190?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14499978#comment-14499978 ]
Sergio Peña commented on HIVE-10190: ------------------------------------ Hey [~sircodesalot] The tests are failing due to the patch attached, and it is because the test output changes. Could you take a look at those tests? > CBO: AST mode checks for TABLESAMPLE with > AST.toString().contains("TOK_TABLESPLITSAMPLE") > ----------------------------------------------------------------------------------------- > > Key: HIVE-10190 > URL: https://issues.apache.org/jira/browse/HIVE-10190 > Project: Hive > Issue Type: Bug > Components: CBO > Affects Versions: 1.2.0 > Reporter: Gopal V > Assignee: Reuben Kuhnert > Priority: Trivial > Labels: perfomance > Attachments: HIVE-10190-querygen.py, HIVE-10190.01.patch, > HIVE-10190.02.patch, HIVE-10190.03.patch, HIVE-10190.04.patch, > HIVE-10190.05.patch, HIVE-10190.05.patch > > > {code} > public static boolean validateASTForUnsupportedTokens(ASTNode ast) { > String astTree = ast.toStringTree(); > // if any of following tokens are present in AST, bail out > String[] tokens = { "TOK_CHARSETLITERAL", "TOK_TABLESPLITSAMPLE" }; > for (String token : tokens) { > if (astTree.contains(token)) { > return false; > } > } > return true; > } > {code} > This is an issue for a SQL query which is bigger in AST form than in text > (~700kb). -- This message was sent by Atlassian JIRA (v6.3.4#6332)