mqliang commented on pull request #6710:
URL: https://github.com/apache/incubator-pinot/pull/6710#issuecomment-807918035


   > 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.
   
   @Jackie-Jiang No. Now we put it at the end of the data table. In V3,  layout 
of data table looks like:
   
   ```
   version number
   num_rows
   num_columns
   exception_start_offset, exception_length
   dictionary_map_start_offset, dictionary_map_length
   data_schema_start_offset, data_scheme_length
   fixed_size_data_start_offset, fixed_size_length
   variable_size_data_start_offset, variable_size_length
   trailer_start_offset, trailer_length
   exception_bytes
   dictionary_map_bytes
   data_schema_bytes
   fixed_size_data_bytes
   variable_size_data_bytes
   trailer_bytes
   ```
   But I agree to name it as metadata.


-- 
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