You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@kyuubi.apache.org by GitBox <gi...@apache.org> on 2021/08/16 15:37:21 UTC

[GitHub] [incubator-kyuubi] yaooqinn edited a comment on pull request #936: Add date info for json event path

yaooqinn edited a comment on pull request #936:
URL: https://github.com/apache/incubator-kyuubi/pull/936#issuecomment-899608447


   I have not caught up with this idea，so let me ask some general questions.
   
   1. Why need we duplicate the date info we already have in all kinds of events？
   2. How does it possible to let event be the sub partition column of date，while they contains totally different schemas
   3. Why does itonly affect the json writer？
   4. How a static date handle session or statement events that might be created in the same engine but in different days？


-- 
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@kyuubi.apache.org

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