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

James Dyer edited comment on SOLR-10617 at 5/11/17 3:47 PM:
------------------------------------------------------------

[~joel.bernstein]  Please review the updated patch.  With a small refactoring 
we can avoid all the copy/paste.  If this is in line with what you'd want, do 
you think it would make sense to go one step further and combine 
CalciteJDBCStream with SqlHandler#SqlHandlerStream and locate this in solr-core?


was (Author: jdyer):
[~joel.bernstein]  Please review the updated patch.  With a small refactoring 
we can avoid all the copy/paste.  If this is in line with what you'd want, do 
you think it would make sense to go one step further and combine 
CalciteJDBCStream with SqlHandler#SqlHandlerStream and locate this in solr-core.

> JDBCStream: support more data types
> -----------------------------------
>
>                 Key: SOLR-10617
>                 URL: https://issues.apache.org/jira/browse/SOLR-10617
>             Project: Solr
>          Issue Type: Improvement
>      Security Level: Public(Default Security Level. Issues are Public) 
>          Components: SolrJ
>            Reporter: James Dyer
>            Assignee: James Dyer
>            Priority: Minor
>         Attachments: SOLR-10617.patch, SOLR-10617.patch, SOLR-10617.patch, 
> SOLR-10617.patch
>
>
> It would be nice if JDBCStream could support Decimal types as well as 
> Timestamp-related types, and CLOBs.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to