GitHub user hsyuan opened a pull request:

    https://github.com/apache/incubator-hawq/pull/991

    HAWQ-1131. Fix error message difference between Orca and Planner

    - In ORCA, due to the way exception handled previously we do warning first
    and then later print error referring that message. In this commit, we
    enhanced the exception handling so we just print a single error message.
    - Removed 'PQO unable to generate a plan' or 'Aborting PQO plan
    generation' message and make the error message as close as the planner.
    - Updated error message with filename and line number from where the 
exception
    is raised.
    - Updated regression test expected output files
    
    Patches are ported the following commits from GPDB:
    
https://github.com/greenplum-db/gpdb/commit/4eb5db7a6bc8ac3faeca761193dc37e2c70e1ea3
    
https://github.com/greenplum-db/gpdb/commit/d41ebde141104dd3db6fa29ac46b4096ee7d4825
    
    Author: Karthikeyan Jambu Rajaraman <karthi....@gmail.com>

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

    $ git pull https://github.com/hsyuan/incubator-hawq master

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

    https://github.com/apache/incubator-hawq/pull/991.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 #991
    
----
commit 339c6102750f21c3d7762505b5da134b51244207
Author: Haisheng Yuan <hy...@pivotal.io>
Date:   2016-11-01T07:27:24Z

    HAWQ-1131. Fix error message difference between Orca and Planner
    
    - In ORCA, due to the way exception handled previously we do warning first
    and then later print error referring that message. In this commit, we
    enhanced the exception handling so we just print a single error message.
    - Removed 'PQO unable to generate a plan' or 'Aborting PQO plan
    generation' message and make the error message as close as the planner.
    - Updated error message with filename and line number from where the 
exception
    is raised.
    - Updated regression test expected output files
    
    Patches are ported the following commits from GPDB:
    
https://github.com/greenplum-db/gpdb/commit/4eb5db7a6bc8ac3faeca761193dc37e2c70e1ea3
    
https://github.com/greenplum-db/gpdb/commit/d41ebde141104dd3db6fa29ac46b4096ee7d4825
    
    Author: Karthikeyan Jambu Rajaraman <karthi....@gmail.com>

----


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to