vogievetsky commented on a change in pull request #11085:
URL: https://github.com/apache/druid/pull/11085#discussion_r611250604
##########
File path: web-console/e2e-tests/tutorial-batch.spec.ts
##########
@@ -195,6 +191,7 @@ async function validateQuery(page: playwright.Page,
datasourceName: string) {
/* page */ 'Talk:Oswald Tilghman',
/* regionIsoCode */ 'null',
/* regionName */ 'null',
+ /* time */ '2015-09-12T00:46:58.771Z',
Review comment:
I updated the e2e test and realized/was faced with this interesting
side-effect:
If you specify the time column via the timestampSpec then that column gets
excluded from the dimension auto-detection system. If instead you use a
transform to construct a `__time` dimension then (obviously) the component
dimensions do not get removed from the auto detection. In this case I set
`__time` to be `timestamp_parse("time") + 1` and now clicking though all the
defaults yields an extra field, the original time column.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]