mapleFU commented on code in PR #557:
URL: https://github.com/apache/parquet-format/pull/557#discussion_r3619574204


##########
Encodings.md:
##########
@@ -38,14 +38,14 @@ For details on current implementation status, see the 
[Implementation Status](ht
 | [Delta-length byte array](#DELTALENGTH)          | DELTA_LENGTH_BYTE_ARRAY = 
6                               | BYTE_ARRAY                                    
    |
 | [Delta Strings](#DELTASTRING)                    | DELTA_BYTE_ARRAY = 7      
                                | BYTE_ARRAY, FIXED_LEN_BYTE_ARRAY              
    |
 | [Byte Stream Split](#BYTESTREAMSPLIT)            | BYTE_STREAM_SPLIT = 9     
                                | INT32, INT64, FLOAT, DOUBLE, 
FIXED_LEN_BYTE_ARRAY |
+| [ALP](#ALP)                                      | ALP = 10                  
                                | FLOAT, DOUBLE                                 
    |

Review Comment:
   What about FLBA and float16 datatype? Or should we even support fp8 or other 
types in the future? ( This can be not in this pr, just curious that)



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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to