[jira] [Commented] (SOLR-2714) JsonLoader does not handle null fields

2011-08-18 Thread JIRA 

[ 
https://issues.apache.org/jira/browse/SOLR-2714?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13086888#comment-13086888
 ] 

Trygve Laugstøl commented on SOLR-2714:
---

Just dropping them sounds good.

 JsonLoader does not handle null fields
 --

 Key: SOLR-2714
 URL: https://issues.apache.org/jira/browse/SOLR-2714
 Project: Solr
  Issue Type: Bug
Affects Versions: 3.3
Reporter: Trygve Laugstøl

 The parser in JsonLoader does not handle null fields when adding a document 
 over http+json.
 Given this document:
 {code}
 [{
   timestamp:2011-08-17T14:11:49.201Z,
   correlationId:N44YFGSQNC,
   logType:event,
   short:Invalidating session: 4zy6cvdtmvu1erlay0sn6rhz,
   long:null
 }]
 {code}
 I'm getting a response code=400 and the error message should finish doc 
 first in the logs.
 It seems that JsonLoader is missing case for JSONParser.NULL in the parser 
 even switch.
 * 
 https://svn.apache.org/repos/asf/lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/JsonLoader.java
 * 
 https://svn.apache.org/repos/asf/labs/noggit/src/main/java/org/apache/noggit/JSONParser.java

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira



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

[jira] [Commented] (SOLR-2714) JsonLoader does not handle null fields

2011-08-17 Thread Yonik Seeley (JIRA) 

[ 
https://issues.apache.org/jira/browse/SOLR-2714?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13086360#comment-13086360
 ] 

Yonik Seeley commented on SOLR-2714:


Hmmm, while the JSON loader never advertised the ability to handle null values, 
there's probably no reason not to do so.
I guess we should just drop null values.

 JsonLoader does not handle null fields
 --

 Key: SOLR-2714
 URL: https://issues.apache.org/jira/browse/SOLR-2714
 Project: Solr
  Issue Type: Bug
Affects Versions: 3.3
Reporter: Trygve Laugstøl

 The parser in JsonLoader does not handle null fields when adding a document 
 over http+json.
 Given this document:
 {code}
 [{
   timestamp:2011-08-17T14:11:49.201Z,
   correlationId:N44YFGSQNC,
   logType:event,
   short:Invalidating session: 4zy6cvdtmvu1erlay0sn6rhz,
   long:null
 }]
 {code}
 I'm getting a response code=400 and the error message should finish doc 
 first in the logs.
 It seems that JsonLoader is missing case for JSONParser.NULL in the parser 
 even switch.
 * 
 https://svn.apache.org/repos/asf/lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/JsonLoader.java
 * 
 https://svn.apache.org/repos/asf/labs/noggit/src/main/java/org/apache/noggit/JSONParser.java

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira



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