[ 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