You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by "ksnijjer (via GitHub)" <gi...@apache.org> on 2023/04/11 20:46:08 UTC

[GitHub] [pinot] ksnijjer opened a new issue, #10591: Add support for DATETIMECONVERT() with Timestamp index

ksnijjer opened a new issue, #10591:
URL: https://github.com/apache/pinot/issues/10591

   Current automatic query rewriting for TimeStamp Index only works for DateTrunc() function, filing this ticket to track enhancing it further to support other common UDF's such as DateTimeConvert(). cc @mayankshriv @Jackie-Jiang 


-- 
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: commits-unsubscribe@pinot.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org

Re: [I] Add support for DATETIMECONVERT() with Timestamp index [pinot]

Posted by "dd-willgan (via GitHub)" <gi...@apache.org>.
dd-willgan commented on issue #10591:
URL: https://github.com/apache/pinot/issues/10591#issuecomment-2030771578

   Ended up finding a workaround for this internally, likely won't be able to get to it for now 😕 .


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org

[GitHub] [pinot] dd-willgan commented on issue #10591: Add support for DATETIMECONVERT() with Timestamp index

Posted by "dd-willgan (via GitHub)" <gi...@apache.org>.
dd-willgan commented on issue #10591:
URL: https://github.com/apache/pinot/issues/10591#issuecomment-1504238702

   I can try to take a stab at this 🙂 


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org

Re: [I] Add support for DATETIMECONVERT() with Timestamp index [pinot]

Posted by "kishoreg (via GitHub)" <gi...@apache.org>.
kishoreg commented on issue #10591:
URL: https://github.com/apache/pinot/issues/10591#issuecomment-2032801942

   @ksnijjer datetrunc seems to be more closer to sql syntax and we probably want to push the users towards datetrunc. can you list the scenarios where datetimeconvert is useful


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org

[GitHub] [pinot] Jackie-Jiang commented on issue #10591: Add support for DATETIMECONVERT() with Timestamp index

Posted by "Jackie-Jiang (via GitHub)" <gi...@apache.org>.
Jackie-Jiang commented on issue #10591:
URL: https://github.com/apache/pinot/issues/10591#issuecomment-1504202920

   cc @xiangfu0 


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org

Re: [I] Add support for DATETIMECONVERT() with Timestamp index [pinot]

Posted by "ksnijjer (via GitHub)" <gi...@apache.org>.
ksnijjer commented on issue #10591:
URL: https://github.com/apache/pinot/issues/10591#issuecomment-2032782598

   @npawar this would be a very useful feature, anyone else can pick this one up?


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org

Re: [I] Add support for DATETIMECONVERT() with Timestamp index [pinot]

Posted by "ksnijjer (via GitHub)" <gi...@apache.org>.
ksnijjer commented on issue #10591:
URL: https://github.com/apache/pinot/issues/10591#issuecomment-2032983419

   @kishoreg I have seen dateTimeConvert() in general been more widely used vs dateTrunc() overall, likely because former is more flexible w.r.t different output formats etc. Secondly in the past we have seen third party tools like Superset auto-rewrite queries using dateTimeConvert() only, as such user can't override it on their end - in which case Timestamp index won't work. 
   
   This aks came from a discussion with @dd-willgan , so perhaps he can share more details too.


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org

Re: [I] Add support for DATETIMECONVERT() with Timestamp index [pinot]

Posted by "ksnijjer (via GitHub)" <gi...@apache.org>.
ksnijjer commented on issue #10591:
URL: https://github.com/apache/pinot/issues/10591#issuecomment-1995111740

   @dd-willgan would be great if you can get this one squared away.


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org