Github user huor commented on the issue:
https://github.com/apache/incubator-hawq/pull/789
+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
Github user linwen commented on the issue:
https://github.com/apache/incubator-hawq/pull/789
The reason is
test cases for aggregates haven't been moved to new feature test framework
yet.
So I add this test case in old installcheck.
And all the test cases for aggregates
Github user changleicn commented on the issue:
https://github.com/apache/incubator-hawq/pull/789
why do we still add the old installcheck tests?
---
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
Github user armenatzoglou commented on the issue:
https://github.com/apache/incubator-hawq/pull/789
LGTM.
---
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,
Github user xunzhang commented on the issue:
https://github.com/apache/incubator-hawq/pull/789
+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