[ 
https://issues.apache.org/jira/browse/TRAFODION-1562?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15857038#comment-15857038
 ] 

ASF GitHub Bot commented on TRAFODION-1562:
-------------------------------------------

Github user DaveBirdsall commented on a diff in the pull request:

    https://github.com/apache/incubator-trafodion/pull/946#discussion_r99955662
  
    --- Diff: core/sql/executor/ExHbaseIUD.cpp ---
    @@ -1022,9 +1088,10 @@ ExWorkProcRetcode 
ExHbaseAccessUpsertVsbbSQTcb::work()
            if (NOT hbaseAccessTdb().computeRowsAffected())
              matches_ = 0;
     
    -       if ((step_ == DONE) &&
    +             if ((step_ == DONE) &&
    --- End diff --
    
    Was this indentation change intentional? It now looks like the "if ((step_ 
== DONE)..." is executed when "NOT hbaseAccessTdb().computeRowsAffected" is 
true.


> Changes in query tree when the upsert command is transformed into merge
> -----------------------------------------------------------------------
>
>                 Key: TRAFODION-1562
>                 URL: https://issues.apache.org/jira/browse/TRAFODION-1562
>             Project: Apache Trafodion
>          Issue Type: Sub-task
>          Components: sql-cmp
>            Reporter: Selvaganesan Govindarajan
>            Assignee: Sandhya Sundaresan
>         Attachments: BatchUpsertTransformation.pdf
>
>
> to improve the performance as explained in the main JIRA



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to