kbendick opened a new pull request, #4668:
URL: https://github.com/apache/iceberg/pull/4668

   Adds toJson / fromJson implementations for two more implementations of 
`MetadataUpdate`:
   1. `AddPartitionSpec`
   2. `AddSortOrder`
   
   Following up on PR feedback from 
https://github.com/apache/iceberg/pull/4632, this PR also renames the 
`writeAs***` and `readAs***` by dorpping the `As` so that the functions read 
better.
   
   I've also placed `AddPartitionSpec` into the switch statement based on its 
declaration order in the `MetadataUpdate` file. As it was skipped in the last 
PR, I moved it down so it's now moved back up.


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