Github user huor commented on the pull request:
https://github.com/apache/incubator-hawq/pull/327#issuecomment-179691649
It is better to set the pointers to NULLs after pfree.
LGTM. +1
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user ztao1987 commented on the pull request:
https://github.com/apache/incubator-hawq/pull/327#issuecomment-179693064
Sure, set it to NULL in the code
---
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
GitHub user ztao1987 opened a pull request:
https://github.com/apache/incubator-hawq/pull/327
HAWQ-390. Fix memory leak in dispatcher.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ztao1987/incubator-hawq HAWQ-390