Closed acezen closed 3 months ago
hi, @lixueclaire, can you help review this PR. This feature would not affect other data type and I would like to merge the implementation first and refactor the testing data of ldbc dataset(use the full ldbc dataset as testing), and add unit tests in another pull request.
Thank you for incorporating these two new data types, which significantly enhance GraphAr's versatility. It would be highly beneficial to include some illustrative examples and ensure that the documentation is updated accordingly to reflect these additions.
Thanks, I will open a new issue to include document and unit tests
Proposed changes
This change add feature that support
Date
andTimestamp
data type in C++ library.Checklist
Further comments
close #398 close #399