tsungchih commented on PR #7498: URL: https://github.com/apache/gravitino/pull/7498#issuecomment-3019162866
> Is it possible that we can add some tests to cover the serialization (DTO -> JSON) and deserialization (JSON -> DTO)? @jerryshao Thanks for the comments. Before being able to implement the serdes of column default values, we need to implement the related DTO classes (such as LiteralDTO, FieldReferenceDTO, FuncExpressionDTO, and UnparsedExpressionDTO). The LiteralDTO has been created in [the previous PR](https://github.com/apache/gravitino/pull/7357) as it is not able to be separated from implementation of `FunctionArg` and of `ColumnDTO`. And the rest of DTO classes were included in this PR. After then, the serdes corresponding to the column default values will be implemented in the next PR. Then the unit tests will be added accordingly. -- 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]
