[
https://issues.apache.org/jira/browse/DRILL-7412?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16955421#comment-16955421
]
ASF GitHub Bot commented on DRILL-7412:
---------------------------------------
paul-rogers commented on pull request #1876: DRILL-7412: Minor unit test
improvements
URL: https://github.com/apache/drill/pull/1876
Many tests intentionally trigger errors. A debug-only log setting
sent those errors to stdout. The resulting stack dumps simply cluttered
the test output, so disabled error output to the console.
Drill can apply bounds checks to vectors. Tests run via Maven
enable bounds checking. Now, bounds checking is also enabled in
"debug mode" (when assertions are enabled, as in an IDE.)
Drill contains two test frameworks. The older BaseTestQuery was
marked as deprecated, but many tests still use it and are unlikely
to be changed soon. So, removed the deprecated marker to reduce the
number of spurious warnings.
Also includes a number of minor clean-ups.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
> Minor unit test improvements
> ----------------------------
>
> Key: DRILL-7412
> URL: https://issues.apache.org/jira/browse/DRILL-7412
> Project: Apache Drill
> Issue Type: Improvement
> Reporter: Paul Rogers
> Assignee: Paul Rogers
> Priority: Minor
>
> Many tests intentionally trigger errors. A debug-only log setting sent those
> errors to stdout. The resulting stack dumps simply cluttered the test output,
> so disabled error output to the console.
> Drill can apply bounds checks to vectors. Tests run via Maven enable bounds
> checking. Now, bounds checking is also enabled in "debug mode" (when
> assertions are enabled, as in an IDE.)
> Drill contains two test frameworks. The older BaseTestQuery was marked as
> deprecated, but many tests still use it and are unlikely to be changed soon.
> So, removed the deprecated marker to reduce the number of spurious warnings.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
