samperson1997 commented on a change in pull request #1721:
URL: https://github.com/apache/iotdb/pull/1721#discussion_r532004993



##########
File path: server/src/main/java/org/apache/iotdb/db/writelog/io/LogWriter.java
##########
@@ -38,16 +38,34 @@
   private FileOutputStream fileOutputStream;
   private FileChannel channel;
   private CRC32 checkSummer = new CRC32();
-  private IoTDBConfig config = IoTDBDescriptor.getInstance().getConfig();
   private ByteBuffer lengthBuffer = ByteBuffer.allocate(4);
   private ByteBuffer checkSumBuffer = ByteBuffer.allocate(8);
+  private long forcePeriodInMs = 0;
 
-  public LogWriter(String logFilePath) {
+  /**
+   * only used by tests
+   * @param logFilePath
+   * @param forcePeriodInMs
+   * @throws FileNotFoundException
+   */
+  public LogWriter(String logFilePath, long forcePeriodInMs) throws 
FileNotFoundException {

Review comment:
       You can add `@TestOnly` for this method




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to