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

David Smiley commented on SOLR-12290:
-------------------------------------

Hi Mark.  In SOLR-11692 [~millerjeff0] and I took care not to wrap a 
closeShield in SolrDispatchFilter.doFilter when the request continues on thru 
chain.doFilter  (line ~367).  Admittedly there isn't a test for this and I 
forget precisely why it mattered.

Otherwise overall the patch looks good.
* JavabinLoader: you can now inline parseAndLoadDocs

> Improve our servlet stream closing prevention code for users and devs.
> ----------------------------------------------------------------------
>
>                 Key: SOLR-12290
>                 URL: https://issues.apache.org/jira/browse/SOLR-12290
>             Project: Solr
>          Issue Type: Task
>      Security Level: Public(Default Security Level. Issues are Public) 
>            Reporter: Mark Miller
>            Assignee: Mark Miller
>            Priority: Minor
>         Attachments: SOLR-12290.patch, SOLR-12290.patch, SOLR-12290.patch, 
> SOLR-12290.patch
>
>
> Original Summary:
> When you fetch a file for replication we close the request output stream 
> after writing the file which ruins the connection for reuse.
> We can't close response output streams, we need to reuse these connections. 
> If we do, clients are hit will connection problems when they try and reuse 
> the connection from their pool.
> New Summary:
> At some point the above was addressed during refactoring. We should remove 
> these neutered closes and review our close shield code.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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

Reply via email to