GitHub user hsyuan opened a pull request:
https://github.com/apache/incubator-hawq/pull/839
HAWQ-983. Fix bug that minirepro generates wrong dependency order of objects
This patch changes gp_dump_query_oids by traversing the parsed query
tree instead of traversing the query tree struct, which has too
many node type and corner cases to consider. Even it is a little bit
risky to traverse the pased query tree string, we haven't seen any
sign that postgresql upstream is going to change the format.
Add 2 options: relation-oids and function-oids into pg_dump command
line tool to help minirepro dump all the objects in correct order.
In addition, we also fix a minirepro python script bug when column
stats has text type most common values that containing single quote,
minirepro failed to escape that text, which causes sql grammar error
and can't insert that statistics.
Update minirepro to deal with error messages correctly, set PGUSER
as default user, let output file accept relative path.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/hsyuan/incubator-hawq HAWQ-983
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-hawq/pull/839.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 #839
----
commit 3691f5e41ac75185420bf6f03e3a9a9c7910576e
Author: Haisheng Yuan <hy...@pivotal.io>
Date: 2016-08-05T21:50:35Z
HAWQ-983. Fix bug that minirepro generates wrong dependency order of objects
This patch changes gp_dump_query_oids by traversing the parsed query
tree instead of traversing the query tree struct, which has too
many node type and corner cases to consider. Even it is a little bit
risky to traverse the pased query tree string, we haven't seen any
sign that postgresql upstream is going to change the format.
Add 2 options: relation-oids and function-oids into pg_dump command
line tool to help minirepro dump all the objects in correct order.
In addition, we also fix a minirepro python script bug when column
stats has text type most common values that containing single quote,
minirepro failed to escape that text, which causes sql grammar error
and can't insert that statistics.
Update minirepro to deal with error messages correctly, set PGUSER
as default user, let output file accept relative path.
----
---
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.
---
