clintropolis commented on code in PR #19711:
URL: https://github.com/apache/druid/pull/19711#discussion_r3648051793
##########
embedded-tests/src/test/java/org/apache/druid/testing/embedded/catalog/CatalogIngestAndQueryTest.java:
##########
@@ -415,6 +423,123 @@ public void testInsertWithMultiClusteringFromQuery()
);
}
+ /**
+ * Create a table with columns (in declared order, which is the physical
segment order — the clustering column
+ * must be declared first):
+ * <p>
+ * varchar_col2 VARCHAR
+ * __time LONG
+ * varchar_col1 VARCHAR
+ * bigint_col1 BIGINT
+ * <p>
+ * and a clustered base table layout with clustering column {@code
varchar_col2}, plus an aggregate projection
+ * ({@code varchar_col2_daily}: group by varchar_col2, sum bigint_col1), so
ingestion produces clustered (V10)
+ * segments that also carry a segment-wide projection.
+ * <p>
+ * Insert the following data:
+ * <p>
+ * __time, varchar_col1, bigint_col1, float_col1, varchar_col2
+ * 2022-12-26T12:34:56,extra,10,"20",2.0,foo
+ * 2022-12-26T12:34:56,extra,9,"30",2.0,foo
+ * 2022-12-26T12:34:56,extra,8,"40",2.0,foq
+ * 2022-12-26T12:34:56,extra,8,"50",2.0,fop
+ * <p>
+ * 'SELECT * from ##tableName' returns columns in catalog order and rows in
segment order: cluster groups in
Review Comment:
fixed, claude was 'inspired' by the old integration tests usage of template
variables and for some reason thought it would be cool to do something like
that here despite ... it being javadoc, and I missed it. 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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
