Jackie-Jiang commented on pull request #6710: URL: https://github.com/apache/incubator-pinot/pull/6710#issuecomment-807891631
> I name it as TrailerKeys since trailer may not only contains metadata KV pairs, but also contains some other data in future -- a metadata key must be a trailer key, but the opposite is not necessarily true. I am OK to rename it as MetadataKeys if we can accept it to call all data we put into the section in future as metadata. CC @mcvsubbu for more input. @mqliang Everything except for the actual result data can be called metadata IMO. I don't like the term "trailer" because it is not a common term in data world, which can cause confusion. Also, we are not really putting it at the end of the data table, it is in front of the actual result data. -- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
