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

Benedict commented on CASSANDRA-8308:
-------------------------------------

bq.  I'm not sure how that's related to this patch

My bad, misread the patch boundary,

Since we're opening/closing an extra file, it might be worth only performing 
the action if the channel isn't the correct size, since it typically will be 
(so, open channel, if incorrect size close it, open raf, set length, reopen 
channel).

I haven't tested the change to introduce strerror - are you confident of it, 
and have you tested it? Might be sensible to split into its own ticket.

Otherwise LGTM

> Windows: Commitlog access violations on unit tests
> --------------------------------------------------
>
>                 Key: CASSANDRA-8308
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-8308
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Joshua McKenzie
>            Assignee: Joshua McKenzie
>            Priority: Minor
>              Labels: Windows
>             Fix For: 3.0
>
>         Attachments: 8308_v1.txt, 8308_v2.txt
>
>
> We have four unit tests failing on trunk on Windows, all with 
> FileSystemException's related to the SchemaLoader:
> {noformat}
> [junit] Test 
> org.apache.cassandra.db.compaction.DateTieredCompactionStrategyTest FAILED
> [junit] Test org.apache.cassandra.cql3.ThriftCompatibilityTest FAILED
> [junit] Test org.apache.cassandra.io.sstable.SSTableRewriterTest FAILED
> [junit] Test org.apache.cassandra.repair.LocalSyncTaskTest FAILED
> {noformat}
> Example error:
> {noformat}
>     [junit] Caused by: java.nio.file.FileSystemException: 
> build\test\cassandra\commitlog;0\CommitLog-5-1415908745965.log: The process 
> cannot access the file because it is being used by another process.
>     [junit]
>     [junit]     at 
> sun.nio.fs.WindowsException.translateToIOException(WindowsException.java:86)
>     [junit]     at 
> sun.nio.fs.WindowsException.rethrowAsIOException(WindowsException.java:97)
>     [junit]     at 
> sun.nio.fs.WindowsException.rethrowAsIOException(WindowsException.java:102)
>     [junit]     at 
> sun.nio.fs.WindowsFileSystemProvider.implDelete(WindowsFileSystemProvider.java:269)
>     [junit]     at 
> sun.nio.fs.AbstractFileSystemProvider.delete(AbstractFileSystemProvider.java:103)
>     [junit]     at java.nio.file.Files.delete(Files.java:1079)
>     [junit]     at 
> org.apache.cassandra.io.util.FileUtils.deleteWithConfirm(FileUtils.java:125)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to