[jira] [Work logged] (HIVE-26361) Merge statement compile fails if split update is off

["ASF GitHub Bot (Jira)" <ji...@apache.org>]

|  ![](cid:jira-generated-image-avatar-849216a1-46f6-4457-bf0d-1bb36f41707e) |
[ASF GitHub
Bot](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=githubbot)
**added a worklog** on [![Sub-task](cid:jira-generated-image-
avatar-3e84bc47-2e38-4e28-9666-a1a9f73c3f30)
HIVE-26361](https://issues.apache.org/jira/browse/HIVE-26361)  
---|---  
|  
---  
|

kasakrisz opened a new pull request, #3410:  
URL: <https://github.com/apache/hive/pull/3410>

  1.     1.       1. What changes were proposed in this pull request?  
Remove extra `,` character after the virtual column `ROW__ID` when generating
update branch of merge statements.

  1.     1.       1. Why are the changes needed?  
Causes parse exception when parsing the rewritten query when both  
```  
hive.split.update  
hive.merge.split.update  
```  
are false

  1.     1.       1. Does this PR introduce _any_ user-facing change?  
Yes. No exception should be thrown.

  1.     1.       1. How was this patch tested?  
```  
mvn test -Dtest.output.overwrite -DskipSparkTests
-Dtest=TestMiniLlapLocalCliDriver -Dqfile=sqlmerge.q -pl itests/qtest -Pitests  
```

  
  
---  
|  Worklog entry:  |  10m logged by [ASF GitHub
Bot](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=githubbot)  
---|---  
Change By: |  [ASF GitHub
Bot](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=githubbot)  
Remaining Estimate: |  0h  
Time Spent: |  10m  
|  |  [ ![Add Comment](cid:jira-generated-image-static-comment-
icon-b3d3de13-ec53-487d-a239-429acd0b1038)
](https://issues.apache.org/jira/browse/HIVE-26361#add-comment "Add Comment")
|  [Add Comment](https://issues.apache.org/jira/browse/HIVE-26361#add-comment
"Add Comment")  
---|---  
  
|  This message was sent by Atlassian Jira (v8.20.10#820010-sha1:ace47f9) |  |
![Atlassian logo](https://issues.apache.org/jira/images/mail/atlassian-email-
logo.png)  
---