Mailing-List: contact dev-help@madlib.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@madlib.incubator.apache.org
From: njayaram2 <git@git.apache.org>
To: dev@madlib.incubator.apache.org
Reply-To: dev@madlib.incubator.apache.org
Message-ID: <git-pr-55-incubator-madlib@git.apache.org>
Subject: [GitHub] incubator-madlib pull request #55: Path: Handling multiple expressions in pa...
Content-Type: text/plain
Date: Fri,  8 Jul 2016 00:22:54 +0000 (UTC)
archived-at: Fri, 08 Jul 2016 00:23:00 -0000

GitHub user njayaram2 opened a pull request:

    https://github.com/apache/incubator-madlib/pull/55

    Path: Handling multiple expressions in partition_expr

    JIRA: MADLIB-1003
    
    The various expressions in the partition_expr parameter are now
    quoted and renamed. These expressions were not renamed earlier,
    which led to an error when there were multiple expressions in the
    parameter. Renaming them using ' AS ' fixes this issue.
    This commit also includes an update to the install check, which
    now includes multiple expressions in partition_expr.

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

    $ git pull https://github.com/njayaram2/incubator-madlib bugfix/jira_1003

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

    https://github.com/apache/incubator-madlib/pull/55.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 #55
    
----
commit ee8a0a842a41e6d2f2f3db9ad0e77bb1ab82c773
Author: Nandish Jayaram <njayaram@pivotal.io>
Date:   2016-07-08T00:19:00Z

    Path: Handling multiple expressions in partition_expr
    
    JIRA: MADLIB-1003
    
    The various expressions in the partition_expr parameter are now
    quoted and renamed. These expressions were not renamed earlier,
    which led to an error when there were multiple expressions in the
    parameter. Renaming them using ' AS ' fixes this issue.
    This commit also includes an update to the install check, which
    now includes multiple expressions in partition_expr.

----


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