pitrou commented on code in PR #100:
URL: https://github.com/apache/parquet-site/pull/100#discussion_r1946775051


##########
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:
   > 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.
   
   You should then add a note or asterisk to point out that "supported" means 
the physical type is returned.
   



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