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

ASF subversion and git services commented on SOLR-17518:
--------------------------------------------------------

Commit f1050ff766d6a0f30fe2784d70684faec346a70c in solr's branch 
refs/heads/branch_9x from Pierre Salagnac
[ https://gitbox.apache.org/repos/asf?p=solr.git;h=f1050ff766d ]

SOLR-17518: Deprecate UpdateRequest.getXml() and replace it with 
XMLRequestWriter (#3200)


> Refactor out a XmlRequestWriter so that RequestWriter is abstract
> -----------------------------------------------------------------
>
>                 Key: SOLR-17518
>                 URL: https://issues.apache.org/jira/browse/SOLR-17518
>             Project: Solr
>          Issue Type: Task
>          Components: SolrJ
>            Reporter: David Smiley
>            Assignee: Pierre Salagnac
>            Priority: Minor
>              Labels: newdev, pull-request-available
>             Fix For: main (10.0)
>
>          Time Spent: 1h 50m
>  Remaining Estimate: 0h
>
> RequestWriter writes XML; some subclasses write other things.  This is 
> terrible API design; the XML choice should be a subclass, RequestWriter 
> should be abstract (or an interface).
> While we're at this, the XML generation is kind of split into multiple 
> places; it should be consolidated: UpdateRequest & ClientUtils have XML 
> generation methods.  Those methods should move to the new XmlRequestWriter.
> BinaryRequestWriter should probably be final and/or ensure the CBOR one does 
> not subclass it, which is weird since CBOR != "javabin".
> Be sure to note this refactoring in the update notes since SolrJ users will 
> be impacted.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

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

Reply via email to