qiaojialin commented on a change in pull request #1060:
URL: https://github.com/apache/incubator-iotdb/pull/1060#discussion_r414947506



##########
File path: tsfile/src/main/java/org/apache/iotdb/tsfile/write/record/Tablet.java
##########
@@ -46,49 +57,43 @@
    */
   public long[] timestamps;
   /**
-   * each object is a primitive type array, which represents values of one
-   * measurement
+   * each object is a primitive type array, which represents values of one 
measurement
    */
   public Object[] values;
   /**
-   * the number of rows to include in this row batch
+   * the number of rows to include in this tablet
    */
-  public int batchSize;
+  public int rowSize;
   /**
    * the maximum number of rows for this row batch
    */
-  private int maxBatchSize;
-
-  /**
-   * total byte size that values occupies
-   */
-  private int valueOccupation = -1;
+  private int maxRowNumber;
 
   /**
    * Return a row batch with default specified row number. This is the standard
-   * constructor (all RowBatch should be the same size).
+   * constructor (all Tablet should be the same size).
    *
    * @param deviceId   the name of the device specified to be written in
    * @param timeseries the list of measurement schemas for creating the row 
batch
    */
-  public RowBatch(String deviceId, List<MeasurementSchema> timeseries) {
+  public Tablet(String deviceId, List<MeasurementSchema> timeseries) {
     this(deviceId, timeseries, DEFAULT_SIZE);
   }
 
   /**
    * Return a row batch with the specified number of rows (maxBatchSize). Only
-   * call this constructor directly for testing purposes. RowBatch should 
normally
+   * call this constructor directly for testing purposes. Tablet should 
normally

Review comment:
       fixed




----------------------------------------------------------------
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:
[email protected]


Reply via email to