[ 
https://issues.apache.org/jira/browse/HIVE-26267?focusedWorklogId=777343&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-777343
 ]

ASF GitHub Bot logged work on HIVE-26267:
-----------------------------------------

                Author: ASF GitHub Bot
            Created on: 02/Jun/22 08:27
            Start Date: 02/Jun/22 08:27
    Worklog Time Spent: 10m 
      Work Description: deniskuzZ commented on code in PR #3325:
URL: https://github.com/apache/hive/pull/3325#discussion_r887702617


##########
standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/TxnHandler.java:
##########
@@ -3712,25 +3712,26 @@ public CompactionResponse compact(CompactionRequest 
rqst) throws MetaException {
             
TxnUtils.createValidCompactWriteIdList(getValidWriteIds(request).getTblValidWriteIds().get(0));
         LOG.debug("ValidCompactWriteIdList: " + 
tblValidWriteIds.writeToString());
 
-        List<String> params = new ArrayList<>();
         StringBuilder sb = new StringBuilder("SELECT \"CQ_ID\", \"CQ_STATE\" 
FROM \"COMPACTION_QUEUE\" WHERE").
           append(" (\"CQ_STATE\" IN(").
             
append(quoteChar(INITIATED_STATE)).append(",").append(quoteChar(WORKING_STATE)).
             append(") OR (\"CQ_STATE\" = 
").append(quoteChar(READY_FOR_CLEANING)).
             append(" AND \"CQ_HIGHEST_WRITE_ID\" = ?))").
             append(" AND \"CQ_DATABASE\"=?").
             append(" AND \"CQ_TABLE\"=?").append(" AND ");
-        params.add(Long.toString(tblValidWriteIds.getHighWatermark()));
-        params.add(rqst.getDbname());
-        params.add(rqst.getTablename());
         if(rqst.getPartitionname() == null) {
           sb.append("\"CQ_PARTITION\" is null");
         } else {
           sb.append("\"CQ_PARTITION\"=?");

Review Comment:
   no need to check explicitly for null,  you can use `pstmt.setObject()`





Issue Time Tracking
-------------------

    Worklog Id:     (was: 777343)
    Time Spent: 40m  (was: 0.5h)

> Addendum to HIVE-26107: perpared statement is not working on Postgres
> ---------------------------------------------------------------------
>
>                 Key: HIVE-26267
>                 URL: https://issues.apache.org/jira/browse/HIVE-26267
>             Project: Hive
>          Issue Type: Bug
>            Reporter: László Végh
>            Assignee: László Végh
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> The assembled prepared statement in
> {code:java}
> org.apache.hadoop.hive.metastore.txn.TxnHandler#compact{code}
> does not work for Postgres DB.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

Reply via email to