rdblue commented on code in PR #461:
URL: https://github.com/apache/parquet-format/pull/461#discussion_r1815652849


##########
VariantShredding.md:
##########
@@ -23,7 +23,7 @@
 > **This specification is still under active development, and has not been 
 > formally adopted.**
 
 The Variant type is designed to store and process semi-structured data 
efficiently, even with heterogeneous values.
-Query engines encode each Variant value in a self-describing format, and store 
it as a group containing `value` and `metadata` binary fields in Parquet.
+Query engines encode each Variant value in a self-describing format, and store 
it as a group containing required `value` and `metadata` binary fields in 
Parquet.
 Since data is often partially homogenous, it can be beneficial to extract 
certain fields into separate Parquet columns to further improve performance.
 We refer to this process as **shredding**.
 Each Parquet file remains fully self-describing, with no additional metadata 
required to read or fully reconstruct the Variant data from the file.

Review Comment:
   I trimmed the intro quite a bit.



-- 
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: [email protected]

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