trafodion-codereview mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From DaveBirdsall <...@git.apache.org>
Subject [GitHub] incubator-trafodion pull request #519: [TRAFODION-1585] Allow MDAM on inner ...
Date Thu, 02 Jun 2016 22:11:26 GMT
GitHub user DaveBirdsall opened a pull request:

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

    [TRAFODION-1585] Allow MDAM on inner child of NJ sometimes

    If it is known that there will be exactly one probe of the inner child of a nested join,
this change enables consideration of MDAM for that inner child.
    
    This is a replacement of https://github.com/apache/incubator-trafodion/pull/174. It takes
the suggestion from @zellerh to use the input logical properties instead of adding constraint
information to the input physical properties. The result is a much simpler code change. I
will close the older pull request shortly.


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

    $ git pull https://github.com/DaveBirdsall/incubator-trafodion Trafodion1585

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

    https://github.com/apache/incubator-trafodion/pull/519.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 #519
    
----
commit e2ffd9e096a841b8e57fc68776c981c3a98816e1
Author: Dave Birdsall <dbirdsall@apache.org>
Date:   2016-06-02T22:07:37Z

    [TRAFODION-1585] Allow MDAM on inner child of NJ if there is only one NJ probe

----


---
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