You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@drill.apache.org by "Paul Rogers (JIRA)" <ji...@apache.org> on 2016/12/01 23:14:58 UTC
[jira] [Updated] (DRILL-5092) JDBC driver includes Drill test code
[ https://issues.apache.org/jira/browse/DRILL-5092?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Paul Rogers updated DRILL-5092:
-------------------------------
Description:
The Drill JDBC driver ({{DrillConnectionImpl}} contains test code and links to test code. This should be factored out separately so that production code does not contain test classes or options to enable test features.
In particular:
{code}
import org.apache.drill.exec.util.TestUtilities;
...
private static void makeTmpSchemaLocationsUnique(StoragePluginRegistry pluginRegistry, Properties props) {
{code}
As a result of the above, the class {{TestUtilities}} is included in the {{src/main/java}} directory, not {{src/test/java}} as it should.
was:
The Drill JDBC driver ({{DrillConnectionImpl}} contains test code and links to test code. This should be factored out separately so that production code does not contain test classes or options to enable test features.
In particular:
{code}
import org.apache.drill.exec.util.TestUtilities;
...
private static void makeTmpSchemaLocationsUnique(StoragePluginRegistry pluginRegistry, Properties props) {
{code}
> JDBC driver includes Drill test code
> ------------------------------------
>
> Key: DRILL-5092
> URL: https://issues.apache.org/jira/browse/DRILL-5092
> Project: Apache Drill
> Issue Type: Improvement
> Affects Versions: 1.8.0
> Reporter: Paul Rogers
> Priority: Minor
>
> The Drill JDBC driver ({{DrillConnectionImpl}} contains test code and links to test code. This should be factored out separately so that production code does not contain test classes or options to enable test features.
> In particular:
> {code}
> import org.apache.drill.exec.util.TestUtilities;
> ...
> private static void makeTmpSchemaLocationsUnique(StoragePluginRegistry pluginRegistry, Properties props) {
> {code}
> As a result of the above, the class {{TestUtilities}} is included in the {{src/main/java}} directory, not {{src/test/java}} as it should.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)