GitHub user njayaram2 opened a pull request:
https://github.com/apache/incubator-madlib/pull/51
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/51.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 #51
----
commit 47f18209656c3483c880c3101c96ecf4dae91e52
Author: Nandish Jayaram <[email protected]>
Date: 2016-06-28T21:55:24Z
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 [email protected] or file a JIRA ticket
with INFRA.
---
