Github user ilooner commented on the issue:
https://github.com/apache/drill/pull/940
Yeah this is ready for final review and commit. To do a quick build and
smoke test you can do
```
mvn -T 1C clean install
Github user paul-rogers commented on the issue:
https://github.com/apache/drill/pull/940
@ilooner, thanks for the improvements. Is this PR now ready for final
review and commit?
Also, if all tests are enabled by default, is there a suggestion of a Maven
command that
Github user ilooner commented on the issue:
https://github.com/apache/drill/pull/940
@paul-rogers I have made a few other changes, could you review once more?
---
Github user ilooner commented on the issue:
https://github.com/apache/drill/pull/940
Please merge my other PR https://github.com/apache/drill/pull/923 before
this one. Thanks!
---
Github user ilooner commented on the issue:
https://github.com/apache/drill/pull/940
Thanks @paul-rogers squashed and ready to commit
---
Github user paul-rogers commented on the issue:
https://github.com/apache/drill/pull/940
Thanks for the explanations and corrections.
Please squash your commits to prepare for commit to master.
+1
---
Github user ilooner commented on the issue:
https://github.com/apache/drill/pull/940
@paul-rogers This is an implementation of the Test Categories suggestion
you made on the dev list. It's ready for review.
---
