[teiid-issues] [JBoss JIRA] (TEIID-5473) Make JDBCMetadataProcessor more extensible with protected methods

Ken Geis (JIRA) issues at jboss.org
Mon Sep 17 20:54:00 EDT 2018


    [ https://issues.jboss.org/browse/TEIID-5473?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13634584#comment-13634584 ] 

Ken Geis commented on TEIID-5473:
---------------------------------

I'm using the Oracle driver.

I just sampled (n=1) and found that without {{-DuseFetchSizeWithLongColumn=true}}, it takes 2m48s. With it, it takes 2m11s. I'm not comfortable setting this property application-wide, but I think it could be [set on the individual connection|https://docs.oracle.com/javase/8/docs/api/java/sql/Connection.html#setClientInfo-java.lang.String-java.lang.String-] if I override {{process(..)}} or {{getConnectorMetadata(..)}}.

Yes, I can now see a path for trying a different Oracle-specific method for fetching index information by returning false in {{getIndexInfoForTable}} and then using my own logic in the overridden {{getConnectorMetadata(..)}}. If you have any hesitation in modifying the API per my request, let me test out my hypothesis first.

> Make JDBCMetadataProcessor more extensible with protected methods
> -----------------------------------------------------------------
>
>                 Key: TEIID-5473
>                 URL: https://issues.jboss.org/browse/TEIID-5473
>             Project: Teiid
>          Issue Type: Feature Request
>          Components: JDBC Connector
>    Affects Versions: 11.1
>            Reporter: Ken Geis
>            Assignee: Steven Hawkins
>             Fix For: 11.2
>
>
> I believe I can write a JDBCMetadataProcessor that works faster against my Oracle database. The important part to override is in getIndexes(..). This method is package-private, so I cannot override it. It would be useful to me if the method (and maybe others) were made protected so that they can be overridden.



--
This message was sent by Atlassian JIRA
(v7.5.0#75005)


More information about the teiid-issues mailing list