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

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

GitHub user sandhyasun opened a pull request:

    https://github.com/apache/trafodion/pull/1708

    Fix for [TRAFODION-3192] UNION ALL up and down queue size default value too 
small, or not dynamically adjusted

    Fix for queue sizes for Union, Transpose and Sequence operators.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/sandhyasun/trafodion sandhyas_queue_work

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/trafodion/pull/1708.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1708
    
----
commit a111065fc659d1b82233cd78607820f3a8bd74c6
Author: Sandhya Sundaresan <sandhya@...>
Date:   2018-08-29T04:33:10Z

    Changes to update queue sizes and add dynamic queue resizing to some 
missing operators.
    (cherry picked from commit f26c0f448a6adf5cdefddc74c4ba840b030e8cf5)
    
    Conflicts:
    
        core/sql/generator/Generator.cpp
        core/sql/generator/Generator.h
        core/sql/sqlcomp/nadefaults.cpp

commit 45207a0579cd863757a0cbd3c7bdc0b25da6442e
Author: Sandhya Sundaresan <sandhya@...>
Date:   2018-08-30T21:49:54Z

    Fix to remove minmax unused methods.

----


> UNION ALL up and down queue size default value too small, or not dynamically 
> adjusted
> -------------------------------------------------------------------------------------
>
>                 Key: TRAFODION-3192
>                 URL: https://issues.apache.org/jira/browse/TRAFODION-3192
>             Project: Apache Trafodion
>          Issue Type: Bug
>          Components: sql-exe
>    Affects Versions: any
>            Reporter: Sandhya Sundaresan
>            Assignee: Sandhya Sundaresan
>            Priority: Major
>             Fix For: 2.4
>
>
> The union up queue size is hardcoded in nadefaults.cpp as a start value as 16 
> which
> is very low if large number of rows are being returned. Scan operator has it 
> set to 2048.
> Some operators further adjust this up size from the original value based on 
> estimate of
> the number of rows being returned. We can do that for union codegen method and
> also change the original up queue size to 2048.
>  
> Noticed that Transpose, Sequence also have the same issue and dynamic queue 
> resizing is missing. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to