clintropolis commented on code in PR #18524:
URL: https://github.com/apache/druid/pull/18524#discussion_r2353919226
##########
processing/src/test/java/org/apache/druid/data/input/impl/AggregateProjectionSpecTest.java:
##########
@@ -112,9 +113,15 @@ void testComputeOrdering_granularity()
ColumnType.LONG,
TestExprMacroTable.INSTANCE
);
+ ExpressionVirtualColumn ptEvery10MinLA = new ExpressionVirtualColumn(
+ "ptEvery10MinLA",
+ "timestamp_floor(__time, 'PT10M', null, 'America/Los_Angeles')",
+ ColumnType.LONG,
+ TestExprMacroTable.INSTANCE
+ );
ExpressionVirtualColumn ptEvery10Min = new ExpressionVirtualColumn(
Review Comment:
oh, i guess it wasn't super obvious pt meant pacific time, since it is also
`PT10M` i just assumed it was being oddly verbose about the 10 minutes part 😅
--
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]