tsungchih commented on PR #6903:
URL: https://github.com/apache/gravitino/pull/6903#issuecomment-2849201930
@xunliu @unknowntpo Based on the comments, I made the following changes:
- moved `json_serdes` into `gravitino.api.types`;
- combined `GenericJsonSerializer` and `GenericJsonDeserializer` into one
`JasonSerializable` interface;
- rename `TypeSerializer` as `TypeSerdes` to implement the
`JsonSerializable` interface;
In the end, The `TypeSerdes` will be used by `ColumnDTO` in the future such
as
```python
from gravitino.api.types.json_serdes.type_serdes import TypeSerdes
class ColumnDTO(Column):
_data_type: Type = field(
metadata=config(
field_name="data_type",
encoder=TypeSerdes.serialize,
decoder=TypeSerdes.deserialize,
)
)
```
Please take a look at it when you're available. Thanks!
--
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]
