tsungchih opened a new pull request, #7357:
URL: https://github.com/apache/gravitino/pull/7357

   <!--
   1. Title: [#<issue>] <type>(<scope>): <subject>
      Examples:
        - "[#123] feat(operator): support xxx"
        - "[#233] fix: check null before access result in xxx"
        - "[MINOR] refactor: fix typo in variable name"
        - "[MINOR] docs: fix typo in README"
        - "[#255] test: fix flaky test NameOfTheTest"
      Reference: https://www.conventionalcommits.org/en/v1.0.0/
   2. If the PR is unfinished, please mark this PR as draft.
   -->
   
   ### What changes were proposed in this pull request?
   
   This is the second part (totally 4 planned) of implementation to the 
following classes from Java to support Column and its default value, including:
   
   - ColumnDTO
   - LiteralDTO
   - FunctionArg
   - PartitionUtils
   
   We implemented the above four classes in one single PR since they have tight 
dependency to each other.
   
   **NOTE** that we haven't implemented the serdes for 
`ColumnDTO.default_value` which will be included in the future PR. Its serdes 
now are dummy ones to always serialize `ColumnDTO.default_value` to `None` and 
deserialize `ColiumnDTO.default_value` as `Column.DEFAULT_VALUE_NOT_SET`.
   
   ### Why are the changes needed?
   
   We need to support Column and its default value in python client.
   
   #5202 
   
   ### Does this PR introduce _any_ user-facing change?
   
   No
   
   ### How was this patch tested?
   
   Unit tests


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

Reply via email to