okumin commented on code in PR #46: URL: https://github.com/apache/hive-site/pull/46#discussion_r2074771620
########## content/Development/qtest.md: ########## @@ -194,3 +194,15 @@ Remote debugging with Query File Test is a potent tool for debugging Hive. With ```sh $ mvn -Pitests -pl itests/qtest -Dmaven.surefire.debug test -Dtest=TestMiniLlapLocalCliDriver -Dqfile=alter1.q ``` + +## Tips for Adding New Tests in Hive + +Following are a few rules of thumb that should be followed when adding new test cases in Hive that require the introduction of new query file(s). Of course, these rules should not be applied if they invalidate the purpose of your test to begin with. These are generally helpful in keeping the test queries concise, minimizing the redundancies where possible, and ensuring that cascading failures due to a single test failure do not occur. + +* Instead of creating your own data file for loading into a new table, use existing data from staged tables like `src`. Review Comment: I'm keeping this statement because we might have not achieved consensus. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org
