[jira] [Commented] (DRILL-5241) JDBC proxy driver: Do not put null value in map

["Rahul Challapalli (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/DRILL-5241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15944234#comment-15944234 ] 

Rahul Challapalli commented on DRILL-5241:
------------------------------------------

A performance test probably might be a good way to verify the fix

> JDBC proxy driver: Do not put null value in map
> -----------------------------------------------
>
>                 Key: DRILL-5241
>                 URL: https://issues.apache.org/jira/browse/DRILL-5241
>             Project: Apache Drill
>          Issue Type: Bug
>          Components: Client - JDBC
>    Affects Versions: 1.9.0
>            Reporter: David Haller
>            Priority: Minor
>              Labels: ready-to-commit
>   Original Estimate: 0h
>  Remaining Estimate: 0h
>
> See GitHub pull request: https://github.com/apache/drill/pull/724
> Hello everyone,
> proxyReturnClass is always null, so interfacesToProxyClassesMap will contain null values only. Adding newProxyReturnClass should be correct.
> This bug does not affect functionality, but probably decreases performance because you get "cache misses" all the time.
> Best regards,
> David.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)