wgtmac commented on code in PR #100: URL: https://github.com/apache/parquet-site/pull/100#discussion_r1946609185
########## content/en/docs/File Format/implementationstatus.md: ########## @@ -45,64 +45,63 @@ Implementations: | Data type | C++ | Java | Go | Rust | cuDF | | ----------------------------------------- | ----- | ----- | ----- | ----- | ----- | -| STRING | | | | | ✅ | -| ENUM | | | | | ❌ | -| UUID | | | | | ❌ | -| 8, 16, 32, 64 bit signed and unsigned INT | | | | | ✅ | -| DECIMAL (INT32) | | | | | ✅ | -| DECIMAL (INT64) | | | | | ✅ | -| DECIMAL (BYTE_ARRAY) | | | | | ✅ | -| DECIMAL (FIXED_LEN_BYTE_ARRAY) | | | | | ✅ | -| DATE | | | | | ✅ | -| TIME (INT32) | | | | | ✅ | -| TIME (INT64) | | | | | ✅ | -| TIMESTAMP (INT64) | | | | | ✅ | -| INTERVAL | | | | | ❌ | -| JSON | | | | | ❌ | -| BSON | | | | | ❌ | -| LIST | | | | | ✅ | -| MAP | | | | | ✅ | -| UNKNOWN (always null) | | | | | ✅ | -| FLOAT16 | | | | | ✅ | +| STRING | ✅ | ✅ | | | ✅ | +| ENUM | ❌ | ✅ | | | ❌ | +| UUID | ❌ | ✅ | | | ❌ | +| 8, 16, 32, 64 bit signed and unsigned INT | ✅ | ✅ | | | ✅ | +| DECIMAL (INT32) | ✅ | ✅ | | | ✅ | +| DECIMAL (INT64) | ✅ | ✅ | | | ✅ | +| DECIMAL (BYTE_ARRAY) | ✅ | ✅ | | | ✅ | +| DECIMAL (FIXED_LEN_BYTE_ARRAY) | ✅ | ✅ | | | ✅ | +| DATE | ✅ | ✅ | | | ✅ | +| TIME (INT32) | ✅ | ✅ | | | ✅ | +| TIME (INT64) | ✅ | ✅ | | | ✅ | +| TIMESTAMP (INT64) | ✅ | ✅ | | | ✅ | +| INTERVAL | ✅ | ✅ | | | ❌ | +| JSON | ✅ | ✅ | | | ❌ | +| BSON | ❌ | ✅ | | | ❌ | +| LIST | ✅ | ✅ | | | ✅ | +| MAP | ✅ | ✅ | | | ✅ | +| UNKNOWN (always null) | ✅ | ✅ | | | ✅ | +| FLOAT16 | ✅ | ✅ | | | ✅ | Review Comment: > If we consider FLOAT16 as supported, then all the others are supported as well. Yes, that's why I marked all logical types as supported in the initial commit. Developers are able to implement their own bindings to write and read such types by extending the `parquet-java` library. > Is there value in adding another option here (P perhaps) that means the implementation at least supports the logical type enum value but will return the physical type? I'm not sure. Maybe it is a minimum requirement for implementation? -- 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]
