[ 
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)

Reply via email to