[ https://issues.apache.org/jira/browse/HIVE-16935?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16061587#comment-16061587 ]
Hive QA commented on HIVE-16935: -------------------------------- Here are the results of testing the latest attachment: https://issues.apache.org/jira/secure/attachment/12874309/HIVE-16935.1.patch {color:green}SUCCESS:{color} +1 due to 4 test(s) being added or modified. {color:red}ERROR:{color} -1 due to 13 failed/errored test(s), 10847 tests executed *Failed tests:* {noformat} org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[tez_smb_main] (batchId=150) org.apache.hadoop.hive.cli.TestMiniTezCliDriver.testCliDriver[explainanalyze_2] (batchId=99) org.apache.hadoop.hive.cli.TestPerfCliDriver.testCliDriver[query14] (batchId=233) org.apache.hadoop.hive.cli.TestPerfCliDriver.testCliDriver[query16] (batchId=233) org.apache.hadoop.hive.cli.TestPerfCliDriver.testCliDriver[query23] (batchId=233) org.apache.hadoop.hive.cli.TestPerfCliDriver.testCliDriver[query94] (batchId=233) org.apache.hadoop.hive.cli.TestSparkCliDriver.testCliDriver[union24] (batchId=125) org.apache.hadoop.hive.ql.parse.TestReplicationScenariosAcrossInstances.testBootstrapFunctionReplication (batchId=217) org.apache.hadoop.hive.ql.parse.TestReplicationScenariosAcrossInstances.testCreateFunctionIncrementalReplication (batchId=217) org.apache.hadoop.hive.ql.parse.TestReplicationScenariosAcrossInstances.testCreateFunctionWithFunctionBinaryJarsOnHDFS (batchId=217) org.apache.hive.hcatalog.api.TestHCatClient.testPartitionRegistrationWithCustomSchema (batchId=178) org.apache.hive.hcatalog.api.TestHCatClient.testPartitionSpecRegistrationWithCustomSchema (batchId=178) org.apache.hive.hcatalog.api.TestHCatClient.testTableSchemaPropagation (batchId=178) {noformat} Test results: https://builds.apache.org/job/PreCommit-HIVE-Build/5755/testReport Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/5755/console Test logs: http://104.198.109.242/logs/PreCommit-HIVE-Build-5755/ Messages: {noformat} Executing org.apache.hive.ptest.execution.TestCheckPhase Executing org.apache.hive.ptest.execution.PrepPhase Executing org.apache.hive.ptest.execution.ExecutionPhase Executing org.apache.hive.ptest.execution.ReportingPhase Tests exited with: TestsFailedException: 13 tests failed {noformat} This message is automatically generated. ATTACHMENT ID: 12874309 - PreCommit-HIVE-Build > Hive should strip comments from input before choosing which CommandProcessor > to run. > ------------------------------------------------------------------------------------ > > Key: HIVE-16935 > URL: https://issues.apache.org/jira/browse/HIVE-16935 > Project: Hive > Issue Type: Bug > Reporter: Andrew Sherman > Assignee: Andrew Sherman > Attachments: HIVE-16935.1.patch > > > While using Beeswax, Hue fails to execute statement with following error: > Error while compiling statement: FAILED: ParseException line 3:4 missing > KW_ROLE at 'a' near 'a' line 3:5 missing EOF at '=' near 'a' > {quote} > -- comment > SET a=1; > SELECT 1; > {quote} > The same code works in Beeline and in Impala. > The same code fails in CliDriver > > h2. Background > Hive deals with sql comments (“-- to end of line”) in different places. > Some clients attempt to strip comments. For example BeeLine was recently > enhanced in https://issues.apache.org/jira/browse/HIVE-13864 to strip > comments from multi-line commands before they are executed. > Other clients such as Hue or Jdbc do not strip comments before sending text. > Some tests such as TestCliDriver strip comments before running tests. > When Hive gets a command the CommandProcessorFactory looks at the text to > determine which CommandProcessor should handle the command. In the bug case > the correct CommandProcessor is SetProcessor, but the comments confuse the > CommandProcessorFactory and so the command is treated as sql. Hive’s sql > parser understands and ignores comments, but it does not understand the set > commands usually handled by SetProcessor and so we get the ParseException > shown above. > -- This message was sent by Atlassian JIRA (v6.4.14#64029)