You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tsfile.apache.org by ji...@apache.org on 2024/11/11 04:03:46 UTC

(tsfile) branch rc/1.1 updated (9a0a8bc6 -> 0bc238bb)

This is an automated email from the ASF dual-hosted git repository.

jiangtian pushed a change to branch rc/1.1
in repository https://gitbox.apache.org/repos/asf/tsfile.git


    from 9a0a8bc6 Fix the problem that ChunkMeta of empty Chunk is not processed (#284)
     new 1716a887 add missing field
     new 0bc238bb spotless & remove V4 related test

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .../reader/series/AbstractFileSeriesReader.java    |  2 +
 .../read/reader/TsFileReaderEmptyChunkTest.java    | 99 ----------------------
 2 files changed, 2 insertions(+), 99 deletions(-)


(tsfile) 01/02: add missing field

Posted by ji...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

jiangtian pushed a commit to branch rc/1.1
in repository https://gitbox.apache.org/repos/asf/tsfile.git

commit 1716a887ee309b7d8d7d3a50864a4ed3754f421f
Author: Tian Jiang <jt...@163.com>
AuthorDate: Mon Nov 11 12:02:27 2024 +0800

    add missing field
    
    (cherry picked from commit 984b203f52c66e893587af62a14a56372fb92c31)
---
 .../org/apache/tsfile/read/reader/series/AbstractFileSeriesReader.java  | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/java/tsfile/src/main/java/org/apache/tsfile/read/reader/series/AbstractFileSeriesReader.java b/java/tsfile/src/main/java/org/apache/tsfile/read/reader/series/AbstractFileSeriesReader.java
index edb8c05e..6558cf22 100644
--- a/java/tsfile/src/main/java/org/apache/tsfile/read/reader/series/AbstractFileSeriesReader.java
+++ b/java/tsfile/src/main/java/org/apache/tsfile/read/reader/series/AbstractFileSeriesReader.java
@@ -19,6 +19,7 @@
 
 package org.apache.tsfile.read.reader.series;
 
+import java.util.ArrayList;
 import org.apache.tsfile.file.metadata.IChunkMetadata;
 import org.apache.tsfile.read.common.BatchData;
 import org.apache.tsfile.read.controller.IChunkLoader;
@@ -35,6 +36,7 @@ public abstract class AbstractFileSeriesReader implements IBatchReader {
   protected IChunkLoader chunkLoader;
   protected List<IChunkMetadata> chunkMetadataList;
   protected IChunkReader chunkReader;
+  protected List<String> currentChunkMeasurementNames = new ArrayList<>();
   private int chunkToRead;
 
   protected Filter filter;


(tsfile) 02/02: spotless & remove V4 related test

Posted by ji...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

jiangtian pushed a commit to branch rc/1.1
in repository https://gitbox.apache.org/repos/asf/tsfile.git

commit 0bc238bb1cf2803ae445aebf15060654de707468
Author: Tian Jiang <jt...@163.com>
AuthorDate: Mon Nov 11 12:04:49 2024 +0800

    spotless & remove V4 related test
    
    (cherry picked from commit b93019fba291b900f9b59c82da8baa2e710a43c0)
---
 .../reader/series/AbstractFileSeriesReader.java    |  2 +-
 .../read/reader/TsFileReaderEmptyChunkTest.java    | 99 ----------------------
 2 files changed, 1 insertion(+), 100 deletions(-)

diff --git a/java/tsfile/src/main/java/org/apache/tsfile/read/reader/series/AbstractFileSeriesReader.java b/java/tsfile/src/main/java/org/apache/tsfile/read/reader/series/AbstractFileSeriesReader.java
index 6558cf22..2b121335 100644
--- a/java/tsfile/src/main/java/org/apache/tsfile/read/reader/series/AbstractFileSeriesReader.java
+++ b/java/tsfile/src/main/java/org/apache/tsfile/read/reader/series/AbstractFileSeriesReader.java
@@ -19,7 +19,6 @@
 
 package org.apache.tsfile.read.reader.series;
 
-import java.util.ArrayList;
 import org.apache.tsfile.file.metadata.IChunkMetadata;
 import org.apache.tsfile.read.common.BatchData;
 import org.apache.tsfile.read.controller.IChunkLoader;
@@ -28,6 +27,7 @@ import org.apache.tsfile.read.reader.IBatchReader;
 import org.apache.tsfile.read.reader.IChunkReader;
 
 import java.io.IOException;
+import java.util.ArrayList;
 import java.util.List;
 
 /** Series reader is used to query one series of one tsfile. */
diff --git a/java/tsfile/src/test/java/org/apache/tsfile/read/reader/TsFileReaderEmptyChunkTest.java b/java/tsfile/src/test/java/org/apache/tsfile/read/reader/TsFileReaderEmptyChunkTest.java
index 6c77a7fe..ea9d83cb 100644
--- a/java/tsfile/src/test/java/org/apache/tsfile/read/reader/TsFileReaderEmptyChunkTest.java
+++ b/java/tsfile/src/test/java/org/apache/tsfile/read/reader/TsFileReaderEmptyChunkTest.java
@@ -20,34 +20,21 @@ package org.apache.tsfile.read.reader;
 
 import org.apache.tsfile.constant.TestConstant;
 import org.apache.tsfile.enums.TSDataType;
-import org.apache.tsfile.file.metadata.IDeviceID;
-import org.apache.tsfile.file.metadata.TableSchema;
 import org.apache.tsfile.file.metadata.enums.CompressionType;
 import org.apache.tsfile.file.metadata.enums.TSEncoding;
 import org.apache.tsfile.file.metadata.statistics.LongStatistics;
-import org.apache.tsfile.read.TsFileSequenceReader;
 import org.apache.tsfile.read.common.TimeRange;
-import org.apache.tsfile.read.common.block.TsBlock;
-import org.apache.tsfile.read.controller.CachedChunkLoaderImpl;
-import org.apache.tsfile.read.controller.MetadataQuerierByFileImpl;
-import org.apache.tsfile.read.query.executor.TableQueryExecutor;
-import org.apache.tsfile.read.reader.block.TsBlockReader;
 import org.apache.tsfile.write.chunk.AlignedChunkWriterImpl;
-import org.apache.tsfile.write.record.Tablet;
 import org.apache.tsfile.write.schema.IMeasurementSchema;
 import org.apache.tsfile.write.schema.MeasurementSchema;
 import org.apache.tsfile.write.writer.TsFileIOWriter;
 
 import org.junit.After;
-import org.junit.Assert;
-import org.junit.Test;
 
 import java.io.File;
 import java.io.IOException;
 import java.util.ArrayList;
-import java.util.Arrays;
 import java.util.List;
-import java.util.Random;
 
 public class TsFileReaderEmptyChunkTest {
 
@@ -59,92 +46,6 @@ public class TsFileReaderEmptyChunkTest {
     new File(FILE_PATH).delete();
   }
 
-  @Test
-  public void testReadEmptyChunk() throws IOException {
-    TsFileSequenceReader tsFileSequenceReader = null;
-    TableQueryExecutor tableQueryExecutor = null;
-    final List<String> measurementNames = Arrays.asList("s1", "s2", "s3", "s4");
-    try (TsFileIOWriter writer = new TsFileIOWriter(new File(FILE_PATH))) {
-      final String tableName = "table";
-      registerTableSchema(writer, tableName);
-      generateDevice(writer, tableName, 1, 1, 10);
-      writer.endFile();
-
-      tsFileSequenceReader = new TsFileSequenceReader(FILE_PATH, true, true);
-      tableQueryExecutor =
-          new TableQueryExecutor(
-              new MetadataQuerierByFileImpl(tsFileSequenceReader),
-              new CachedChunkLoaderImpl(tsFileSequenceReader),
-              TableQueryExecutor.TableQueryOrdering.DEVICE);
-      final TsBlockReader tsBlockReader =
-          tableQueryExecutor.query(tableName, measurementNames, null, null, null);
-
-      int nullValueCount = 0;
-
-      while (tsBlockReader.hasNext()) {
-        final TsBlock tsBlock = tsBlockReader.next();
-        final TsBlock.TsBlockRowIterator iterator = tsBlock.getTsBlockRowIterator();
-        while (iterator.hasNext()) {
-          final Object[] row = iterator.next();
-          for (Object o : row) {
-            if (o == null) {
-              nullValueCount++;
-            }
-          }
-        }
-      }
-
-      Assert.assertEquals(10, nullValueCount);
-    } catch (final Exception e) {
-      Assert.fail(e.getMessage());
-    } finally {
-      if (tsFileSequenceReader != null) {
-        tsFileSequenceReader.close();
-      }
-    }
-  }
-
-  private void registerTableSchema(final TsFileIOWriter writer, final String tableName) {
-    final List<IMeasurementSchema> schemas =
-        Arrays.asList(
-            new MeasurementSchema(
-                "id", TSDataType.TEXT, TSEncoding.PLAIN, CompressionType.UNCOMPRESSED),
-            new MeasurementSchema("s1", TSDataType.INT64),
-            new MeasurementSchema("s2", TSDataType.INT64),
-            new MeasurementSchema("s3", TSDataType.INT64),
-            new MeasurementSchema("s4", TSDataType.INT64));
-    final List<Tablet.ColumnType> columnTypes =
-        Arrays.asList(
-            Tablet.ColumnType.ID,
-            Tablet.ColumnType.MEASUREMENT,
-            Tablet.ColumnType.MEASUREMENT,
-            Tablet.ColumnType.MEASUREMENT,
-            Tablet.ColumnType.MEASUREMENT);
-    final TableSchema tableSchema = new TableSchema(tableName, schemas, columnTypes);
-    writer.getSchema().registerTableSchema(tableSchema);
-  }
-
-  private void generateDevice(
-      final TsFileIOWriter writer,
-      final String tableName,
-      final int deviceNum,
-      final int minTime,
-      final int maxTime)
-      throws IOException {
-    for (int i = 0; i < deviceNum; i++) {
-      final IDeviceID deviceID =
-          IDeviceID.Factory.DEFAULT_FACTORY.create(new String[] {tableName, "d" + i});
-      writer.startChunkGroup(deviceID);
-      final List<String> measurementNames = Arrays.asList("s1", "s2", "s3", "s4");
-      generateSimpleAlignedSeriesToCurrentDevice(
-          writer,
-          Arrays.asList("s1", "s2", "s3", "s4"),
-          new TimeRange[] {new TimeRange(minTime, maxTime)},
-          new Random().nextInt(measurementNames.size()));
-      writer.endChunkGroup();
-    }
-  }
-
   public void generateSimpleAlignedSeriesToCurrentDevice(
       final TsFileIOWriter writer,
       final List<String> measurementNames,