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

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

Commit 783f3bc393055c476d4ceb3995412a3868fcfca6 in solr's branch 
refs/heads/branch_10x from Surya Shukla
[ https://gitbox.apache.org/repos/asf?p=solr.git;h=783f3bc3930 ]

SOLR-18183: Permit LockType.NONE for ephemeral directories (#4660)

(cherry picked from commit 8fa14f1048f00decc7ade6fede1350a862e97d7e)


> All EphemeralDirectoryFactory should permit LockType.NONE
> ---------------------------------------------------------
>
>                 Key: SOLR-18183
>                 URL: https://issues.apache.org/jira/browse/SOLR-18183
>             Project: Solr
>          Issue Type: Improvement
>            Reporter: David Smiley
>            Priority: Minor
>              Labels: newdev, pull-request-available
>          Time Spent: 1h 10m
>  Remaining Estimate: 0h
>
> RAMDirectoryFactory and ByteBuffersDirectoryFactory do not permit 
> LockType.NONE -- but let's support that.  MockDirectoryFactory returns NONE 
> with no complaint.  The comment their says it's "unused"... which is curious.
> This change is motivated by the experienced observation that 
> UpdateLog.initTlogDir hard-codes the choice of NONE when it temporarily 
> creates a a Directory to aid in cleaning up its own files.  A test 
> configuring RAMDirectoryFactory & updateLog configured, doesn't work.  
> Granted, it's questionable why an UpdateLog is enabled in the first place 
> when an ephemeral directory is in use.  Arguably (separate issue), perhaps 
> the DirectoryFactory should be responsible for also managing the UpdateLog 
> creation so as to ensure one isn't created when it's not sensible.



--
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