CNDB-15032: set SequentialWriter channel to write-only by default

driftx · driftx · commit 0feb87ffc49b · 2025-07-31T16:21:21.000-05:00
diff --git a/src/java/org/apache/cassandra/io/util/SequentialWriter.java b/src/java/org/apache/cassandra/io/util/SequentialWriter.java
@@ -104,11 +104,6 @@ protected Throwable doAbort(Throwable accumulate)
     }
 
     // TODO: we should specify as a parameter if we permit an existing file or not
-    private static FileChannel openChannel(File file)
-    {
-        return openChannel(file, true);
-    }
-
     private static FileChannel openChannel(File file, boolean readable)
     {
         try
@@ -171,7 +166,7 @@ public SequentialWriter(File file, SequentialWriterOption option)
      */
     public SequentialWriter(File file, SequentialWriterOption option, boolean strictFlushing)
     {
-        this(file, true, option, strictFlushing);
+        this(file, false, option, strictFlushing);
     }
 
     public SequentialWriter(File file, boolean readable, SequentialWriterOption option, boolean strictFlushing)

Original file line number	Diff line number	Diff line change
`@@ -104,11 +104,6 @@ protected Throwable doAbort(Throwable accumulate)`
`104`	`104`	`}`
`105`	`105`
`106`	`106`	`// TODO: we should specify as a parameter if we permit an existing file or not`
`107`		`- private static FileChannel openChannel(File file)`
`108`		`- {`
`109`		`- return openChannel(file, true);`
`110`		`- }`
`111`		`-`
`112`	`107`	`private static FileChannel openChannel(File file, boolean readable)`
`113`	`108`	`{`
`114`	`109`	`try`
`@@ -171,7 +166,7 @@ public SequentialWriter(File file, SequentialWriterOption option)`
`171`	`166`	`*/`
`172`	`167`	`public SequentialWriter(File file, SequentialWriterOption option, boolean strictFlushing)`
`173`	`168`	`{`
`174`		`- this(file, true, option, strictFlushing);`
	`169`	`+ this(file, false, option, strictFlushing);`
`175`	`170`	`}`
`176`	`171`
`177`	`172`	`public SequentialWriter(File file, boolean readable, SequentialWriterOption option, boolean strictFlushing)`