zclllyybb commented on issue #45530: URL: https://github.com/apache/doris/issues/45530#issuecomment-2564379533
> Hi @zclllyybb, there are many compression algorithms implemented in the block_compression.cpp file. Do we want to support multiple compression algorithms for this function with function signature like COMPRESS(String, _compression_algorithm_)? UPDATE: I have gone through your function_uuid PR. I also went through some more function implementations in the code base. I will start my implementation soon. We dont need to support the second argument. There's two reasons: 1. we should keep compatible with MySQL. so that it will be easier for users to migrate 2. these functions aim to compress information. since the compress algorithms don't differ very much. user may not be very clear about the different of algorithm's choice. so support with the only algorithm will be better. -- 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: commits-unsubscr...@doris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org
