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]

Reply via email to