mcvsubbu commented on pull request #6331:
URL: https://github.com/apache/incubator-pinot/pull/6331#issuecomment-741259630
> > > can we store creation time in the tableConfig instead of relying on
the znode in ZK.
> >
> >
> > Is the reasoning here to make it easier to move away from zk? Or, do you
expect that the metadata is somehow changeable in zk? Or, is it that we can
avoid a new API (I like that) and just get the table config?
> > Another way can be to get the metadata along with the `GET` API on
tables?
>
> So Subbu's suggestion for GET api on tables will be something like:
>
> ```
> {
> "OFFLINE" : {
> ...
> },
> "REALTIME" : {
> ...
> },
> "METADATA" : {
> "REALTIME" : {
> "creationTime" : "20201206T2130Z"
> },
> "OFFLINE" : {
> "creationTime" : "20201206T2130Z"
> }
> }
> }
> ```
I like sending out metadata along with the table API. What do you say,
@kishoreg ? This also allows us to add other metadata later on if needed.
I am not sure I prefer including metadata in the table config znode
----------------------------------------------------------------
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]
