rdblue opened a new pull request, #485:
URL: https://github.com/apache/parquet-format/pull/485

   ### Rationale for this change
   
   I've talked with several community members that intend to implement JSON 
conversion, but do not want the conversion specified in the Variant spec to be 
required. If the conversion is not required, I don't think that it makes sense 
to add it to the spec.
   
   The conversion was originally added because we wanted to ensure that engines 
that do not support Variant could interact with the data. At the time, it 
seemed like a good idea to standardize how conversion to JSON took place for 
those engines. But there is necessarily data loss when converting to JSON. It's 
hard to decide just how lossy this conversion (to JSON and back) should be and 
what value there is to minimizing the loss. That's why people don't intend to 
implement a consistent well-defined conversion. Without willingness to follow 
this section of the spec, it should be removed.
   
   ### What changes are included in this PR?
   
   Remove the JSON conversion table and section from the Variant spec.
   
   ### Do these changes have PoC implementations?
   
   No. The conversions in this table were never implemented.


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