[GitHub] incubator-hawq pull request #1115: HAWQ-1324. Fixed crash at query cancel, s...

[liming01]

GitHub user liming01 opened a pull request:

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

HAWQ-1324. Fixed crash at query cancel, signal hanler cannot call uns…

…afe elog()

1) The fix cannot completely fix this kinds of crash, it just reduce the 
possibility of this kind of crash. The complete fix is to move 
ProcessInterrupts() calling out of StatementCancelHandler().
2) The log can be removed because afterward in all 3 branches, hawq will 
print out similar message using ereport().

Thanks.

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

$ git pull https://github.com/liming01/incubator-hawq mli/HAWQ-1324

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

https://github.com/apache/incubator-hawq/pull/1115.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 #1115


commit 6435af41f9dd981dd672637337a1dd7112068e3f
Author: Ming LI 
Date:   2017-02-13T07:16:53Z

HAWQ-1324. Fixed crash at query cancel, signal hanler cannot call unsafe 
elog()




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

[GitHub] incubator-hawq issue #1111: HAWQ-1319. Expand pom.xml file for src/test dire...

[edespino]

Github user edespino commented on the issue:

https://github.com/apache/incubator-hawq/pull/
  
Closing this PR as @huor and team will be updating the pom.xml.  They will 
be adding ASF headers as appropriate to additional source files.

-=e


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

[GitHub] incubator-hawq pull request #1111: HAWQ-1319. Expand pom.xml file for src/te...

[edespino]

Github user edespino closed the pull request at:

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


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

[GitHub] incubator-hawq issue #1111: HAWQ-1319. Expand pom.xml file for src/test dire...

[huor]

Github user huor commented on the issue:

https://github.com/apache/incubator-hawq/pull/
  
@edespino, please close this pr and we will update pom and add license 
header in source file in https://issues.apache.org/jira/browse/HAWQ-1323


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

[GitHub] incubator-hawq issue #1113: HAWQ-1321. failNames wrongly uses memory context...

[paul-guo-]

Github user paul-guo- commented on the issue:

https://github.com/apache/incubator-hawq/pull/1113
  
+1


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

[GitHub] incubator-hawq issue #1113: HAWQ-1321. failNames wrongly uses memory context...

[zhangh43]

Github user zhangh43 commented on the issue:

https://github.com/apache/incubator-hawq/pull/1113
  
+1


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