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

Michael Ryan commented on SOLR-3035:
------------------------------------

Shouldn't this be using writeShort and writeByte for short and byte, instead of 
writeInt?

Looking at 3.6 code... in ByteField, it calls XMLWriter.writeByte(String, 
String), which writes it out in a <byte> tag. However, there are also two 
methods - writeByte(String, byte) and writeShort(String, short) - which are 
using writeInt(), which seems like another bug. On trunk, all of this code is 
gone, so I guess it is using writeInt for all of them?
                
> XML and JSON response writers don't handle short or binary field types
> ----------------------------------------------------------------------
>
>                 Key: SOLR-3035
>                 URL: https://issues.apache.org/jira/browse/SOLR-3035
>             Project: Solr
>          Issue Type: Bug
>          Components: Response Writers
>    Affects Versions: 3.1, 3.5, 4.0-ALPHA
>            Reporter: Ken Krugler
>            Priority: Blocker
>             Fix For: 4.0-ALPHA
>
>         Attachments: distributed-search.zip, ResponseWriterTest.java, 
> SOLR-3035.patch
>
>
> The XML and JSON response writers return data from short schema fields as 
> "java.lang.Short:<value>".
> In addition, binary fields (for sharded requests) are returned as "[B:[B@<hex 
> address of array>", which makes them totally unusable.
> This appears to be due to their writeVal() methods not having cases to handle 
> val instanceof Short and val instance of byte[]

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to