[
https://issues.apache.org/jira/browse/SOLR-18033?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18046023#comment-18046023
]
Chris M. Hostetter commented on SOLR-18033:
-------------------------------------------
generally speaking, our {{FieldType}} api with all of the various & convoluted
methods added over the years (many related but not exactly the same) combined
with all the hardcoded case statements, if statements, and assumptions through
out the solr-core code base make it really hard to imagine what a "safe"
generalized solution to this problem will look like.
> binary based fields can't control their external based representation
> ---------------------------------------------------------------------
>
> Key: SOLR-18033
> URL: https://issues.apache.org/jira/browse/SOLR-18033
> Project: Solr
> Issue Type: Bug
> Reporter: Chris M. Hostetter
> Priority: Major
> Attachments: SOLR-18033.patch
>
>
> This was discovered while working on SOLR-17975:
> {quote}... the main issue i *HAVE* found is that even though FieldType has
> method(s) for converting internal BytesRef's into something type specific
> that can be returned to the client, SolrDocumentFetcher isn't consistently
> using those methods in all useDocValuesAsStored situations ... it's got it's
> own special case statement with specialized conditionals that would need to
> be rethought/refactored to ensure the BINARY DocValues for this new field
> type (and any other similar field types) can be reconstituted back into
> _whatever_ external representation _[...the field type wants to use...]_
> {quote}
> While working on a patch with tests for this, I discovered that the same
> problem exists for stored fields due to hardcoded logic in DocsStreamer --
> even though {{FieldType}} has a distinct {{void
> write(TextResponseWriter,...}}} method (for when the response format requires
> String-ification) distinct from the {{Object toObject(IndexableField)}}
> (designed for use by the javabincodec) {{DocStreamer}} only uses {{toObject}}
> with a hardcoded list of special classes -- otherwise it uses the very old
> {{String toExternal(IndexableField)}} method
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
