BanyanDB: fix when setting @BanyanDB.TimestampColumn, the column should not be indexed. (#13525)

Author: wankai123

docs/en/api/query-protocol.md

@@ -169,9 +169,8 @@ extend type Query {
     queryBasicTraces(condition: TraceQueryCondition, debug: Boolean): TraceBrief
     queryBasicTracesByName(condition: TraceQueryConditionByName, debug: Boolean): TraceBrief
     # Read the specific trace ID with given trace ID
-    queryTrace(traceId: ID!, debug: Boolean): Trace
-    # Only for BanyanDB, can be used to query the trace in the cold stage.
-    queryTraceFromColdStage(traceId: ID!, duration: Duration!, debug: Boolean): Trace
+    # duration is optional, and only for BanyanDB. If not provided, means search in the last 1 day.
+    queryTrace(traceId: ID!, duration: Duration, debug: Boolean): Trace
     # Read the list of searchable keys
     queryTraceTagAutocompleteKeys(duration: Duration!):[String!]
     # Search the available value options of the given key.
@@ -181,6 +180,17 @@ extend type Query {
 
 Trace query fetches trace segment lists and spans of given trace IDs.
 
+### Trace-v2
+```graphql
+extend type Query {
+    queryTraces(condition: TraceQueryCondition, debug: Boolean): TraceList
+    # Feature detection endpoint: returns true if the backend supports the Query Traces V2 API.
+    # Returns false if the backend does not support Query Traces V2.
+    # This field is intended to assist clients in migrating to the new API.
+    hasQueryTracesV2Support: Boolean!
+}
+```
+
 ### Alarm
 ```graphql
 extend type Query {

docs/en/changes/changes.md

@@ -103,6 +103,7 @@
 * Fix potential NPE in the `AlarmStatusQueryHandler`.
 * Aggregate TopN Slow SQL by service dimension.
 * BanyanDB: support add group prefix (namespace) for BanyanDB groups.
+* BanyanDB: fix when setting `@BanyanDB.TimestampColumn`, the column should not be indexed.
 
 #### UI
 

oap-server/server-core/src/main/java/org/apache/skywalking/oap/server/core/analysis/manual/segment/SegmentRecord.java

@@ -98,7 +98,6 @@ public class SegmentRecord extends Record implements BanyanDBTrace {
     @Getter
     @ElasticSearch.EnableDocValues
     @Column(name = START_TIME)
-    @BanyanDB.NoIndexing
     private long startTime;
     @Setter
     @Getter

oap-server/server-core/src/main/java/org/apache/skywalking/oap/server/core/analysis/manual/spanattach/SWSpanAttachedEventRecord.java

@@ -87,7 +87,6 @@ public class SWSpanAttachedEventRecord extends Record implements BanyanDBTrace,
     @Getter
     @ElasticSearch.EnableDocValues
     @Column(name = TIMESTAMP)
-    @BanyanDB.NoIndexing
     private long timestamp;
 
     @Override

oap-server/server-core/src/main/java/org/apache/skywalking/oap/server/core/analysis/manual/spanattach/SpanAttachedEventRecord.java

@@ -87,7 +87,6 @@ public class SpanAttachedEventRecord extends Record implements BanyanDBTrace, Ba
     @Getter
     @ElasticSearch.EnableDocValues
     @Column(name = TIMESTAMP)
-    @BanyanDB.NoIndexing
     private long timestamp;
 
     @Override

oap-server/server-core/src/main/java/org/apache/skywalking/oap/server/core/profiling/ebpf/storage/EBPFProfilingTaskRecord.java

@@ -83,7 +83,6 @@ public class EBPFProfilingTaskRecord extends NoneStream {
     private int targetType = EBPFProfilingTargetType.UNKNOWN.value();
     @ElasticSearch.EnableDocValues
     @Column(name = CREATE_TIME)
-    @BanyanDB.NoIndexing
     private long createTime;
     @Column(name = LAST_UPDATE_TIME)
     private long lastUpdateTime;

oap-server/server-core/src/main/java/org/apache/skywalking/oap/server/core/profiling/trace/ProfileTaskRecord.java

@@ -70,7 +70,6 @@ public StorageID id() {
     private String taskId;
     @ElasticSearch.EnableDocValues
     @Column(name = START_TIME)
-    @BanyanDB.NoIndexing
     private long startTime;
     @Column(name = DURATION)
     private int duration;

oap-server/server-core/src/main/java/org/apache/skywalking/oap/server/core/profiling/trace/ProfileThreadSnapshotRecord.java

@@ -63,7 +63,6 @@ public class ProfileThreadSnapshotRecord extends Record {
     private String segmentId;
     @ElasticSearch.EnableDocValues
     @Column(name = DUMP_TIME)
-    @BanyanDB.NoIndexing
     private long dumpTime;
     @ElasticSearch.EnableDocValues
     @Column(name = SEQUENCE)

oap-server/server-core/src/main/java/org/apache/skywalking/oap/server/core/storage/annotation/BanyanDB.java

@@ -106,7 +106,8 @@
     /**
      * Force disabling indexing declare through {@link Column}.
      * In BanyanDB, some additional conditions could be done in server memory, no indexing required in this case.
-     *
+     * In the Trace model, no indexing means no tag would be created in BanyanDB.
+     * In the Stream model, no indexing means no index rule would be created in BanyanDB.
      * @since 9.1.0
      */
     @Target({ElementType.FIELD})
@@ -148,9 +149,9 @@ enum IndexType {
     }
 
     /**
-     * timestampColumn is to identify which column in {@link Record} is providing the timestamp(millisecond) for
-     * BanyanDB.
+     * timestampColumn is to identify which column in {@link Record} is providing the timestamp(millisecond) for BanyanDB.
      * BanyanDB stream requires a timestamp in milliseconds.
+     * Notice, the timestamp column would not create an index rule in BanyanDB.
      *
      * @since 9.3.0
      */

oap-server/server-core/src/main/java/org/apache/skywalking/oap/server/core/storage/model/ModelColumn.java

@@ -112,6 +112,7 @@ public ModelColumn(ColumnName columnName,
     }
 
     /**
+     * Notice, for BanyanDB, should use {@link BanyanDBExtension#shouldIndex()} instead.
      * @return true means this column should be indexed, as it would be a query condition.
      */
     public boolean shouldIndex() {