trafodion-codereview mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sandhyasun <...@git.apache.org>
Subject [GitHub] trafodion pull request #1712: Follow on to [TRAFODION-3192] that caused one ...
Date Tue, 11 Sep 2018 07:07:43 GMT
GitHub user sandhyasun opened a pull request:

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

    Follow on to [TRAFODION-3192] that caused one regression failure 

    Follow on to TRAFODION-3192 that caused one regression failure on Jenkins alone since
the queue resizing caused the executor/TEST106 that tests cancel to not work as expected.
The upsert subtest was running too fast to get cancelled. So added some CQDs to use smaller
queue sizes and override dynamic queue resizing.Now it runs long enough like it used to to
allow getting  canceled and give expected results.
    Also reverted an inadvertant change to the NJ down queue size that was used for debugging.
Reverted it back to what it was (2048) 
    


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/1712.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 #1712
    
----
commit 859f071e7dd962e2c464461c16cc56a56935b72b
Author: Sandhya Sundaresan <sandhya@...>
Date:   2018-09-11T07:02:08Z

    Follow on to TRAFODION-3192 that caused one regression failure on Jenkins alone since
the queue resizing caused the executor/TEST106 that tests cancel to not work as expected.
The upsert subtest was running too fast to get cancelled. So added some CQds to use smaller
queue sizes and override dynamic queue resizing.Now it runs long enough to get canceled and
give expected results.

----


---

Mime
View raw message