[
https://issues.apache.org/jira/browse/SOLR-8770?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15178363#comment-15178363
]
Shawn Heisey edited comment on SOLR-8770 at 3/3/16 6:52 PM:
------------------------------------------------------------
Is there any acceptable reason to allow a null field value in a
SolrInputDocument? Specifically I'm wondering if we could throw
IllegalArgumentException if a null object is used on methods like addField and
setField.
Fixing what I noticed about the binary writer would still be a good idea.
was (Author: elyograg):
Is there any acceptable reason to allow a null field value in a
SolrInputDocument? Specifically I'm wondering if we could throw
IllegalArgumentException if a null object is used on methods like addField and
setField.
> BinaryRequestWriter interprets null object in field as literal "NULL" string
> ----------------------------------------------------------------------------
>
> Key: SOLR-8770
> URL: https://issues.apache.org/jira/browse/SOLR-8770
> Project: Solr
> Issue Type: Bug
> Components: clients - java
> Affects Versions: 5.5
> Reporter: Shawn Heisey
>
> From what I've been able to determine, if a null object is added with
> SolrInputDocument#addField, the xml writer does not include that field in the
> request, but the binary writer sends the literal string "NULL".
> This became apparent when upgrading SolrJ to 5.5, which uses the binary
> writer by default. Switching back to 5.4.1 fixed it, until I forced the
> 5.4.1 client to use the binary writer. My source data is MySQL. JDBC is
> where the null objects are coming from.
> Adding a null check to my doc.addField call has fixed my program with the 5.5
> client, but this is likely to catch other upgrading users off guard.
> At the very least, the 5.5.1 CHANGES.txt file needs a note, but I believe the
> behavior of the binary writer should match the behavior of the xml writer.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org
