trafodion-codereview mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From zellerh <...@git.apache.org>
Subject [GitHub] incubator-trafodion pull request #1170: [TRAFODION-2628] SQL engine work for...
Date Thu, 06 Jul 2017 23:22:05 GMT
GitHub user zellerh opened a pull request:

    https://github.com/apache/incubator-trafodion/pull/1170

    [TRAFODION-2628] SQL engine work for Elasticity

    It turns out that the compiler already kept a list of node ids,
    not assuming that node ids are consecutively numbered 0...n-1.
    However, this list was contained in lots of other data structures
    that are dead code, coming from the time before Trafodion was
    open-sourced. The change does two things: a) getting rid of obsolete
    info and b) maintining a list of node ids in the NAClusterInfo
    class.
    
    This change does not include executor modifications. The executor
    might work just fine, or we may need to add a fix (or optimization)
    for cases where we pick a node id for a new process at runtime.

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

    $ git pull https://github.com/zellerh/incubator-trafodion bug/elasticity

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

    https://github.com/apache/incubator-trafodion/pull/1170.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 #1170
    
----
commit 4bea445d03b5712007ab326464a8b64b8c2b3740
Author: Hans Zeller <hzeller@apache.org>
Date:   2017-07-06T23:11:55Z

    [TRAFODION-2628] SQL engine work for Elasticity
    
    It turns out that the compiler already kept a list of node ids,
    not assuming that node ids are consecutively numbered 0...n-1.
    However, this list was contained in lots of other data structures
    that are dead code, coming from the time before Trafodion was
    open-sourced. The change does two things: a) getting rid of obsolete
    info and b) maintining a list of node ids in the NAClusterInfo
    class.
    
    This change does not include executor modifications. The executor
    might work just fine, or we may need to add a fix (or optimization)
    for cases where we pick a node id for a new process at runtime.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message