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


##########
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:
   What do we mean by logical types are supported? Is it like we won't throw an 
exception and return something at least, or we expect to actually represent the 
related value correctly?
   Since `parquet-java` does not have its own representation type system 
(except the one in the example package that mainly handles the primitive types 
only), the question is whether at least one binding supports the related type. 
I am not aware that any binding supports `INTERVAL` or `BSON`. `JSON` is mostly 
handle as string without any syntax check.



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