Github user jaltekruse commented on the pull request:
https://github.com/apache/drill/pull/380#issuecomment-185825063
Could you also add result verification? While there are some older tests
that just run queries to verify that errors that previously occurred are gone,
we have been enforcing that new tests since the test builder was added verify
their results. You can use the test builder to add records in a loop to the
expected result set.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
