[ 
https://issues.apache.org/jira/browse/NIFI-2827?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17577616#comment-17577616
 ] 

Joe Witt commented on NIFI-2827:
--------------------------------

Thanks for contributing an example patch.  Adding support for these is fine but 
we'll definitely want the key things related to compression levels included.  
We use specific libraries for specific compression types (or at least had in 
the past) because different libraries gave different trade-offs including for 
things like memory usage/etc..  We'll likely retain that unless research has 
been done to show it is no longer necessary.  In any case adding new 
compression/decompression types is often painless and certain commons compress 
helps a lot there.  So happy to add support for these but we'll need someone to 
write up the full patch and someone to review it.  Truthfully these days 
basically all patches that make it in go through Github/Pull Requests because 
there is some extremely helpful automation that comes along with that to reduce 
the review timeline.

Thanks

> CompressContent should support brotli (MIT) and zstd (BSD)
> ----------------------------------------------------------
>
>                 Key: NIFI-2827
>                 URL: https://issues.apache.org/jira/browse/NIFI-2827
>             Project: Apache NiFi
>          Issue Type: New Feature
>            Reporter: Andre F de Miranda
>            Priority: Major
>         Attachments: NIFI-2827-zstd.patch
>
>
> brotli and zstd are gaining lots of momentum across the IoT sphere and I 
> suspect it would be a nice to have in NiFi to be able to support those two 
> emerging compression algorithms.
> Both brotli and zstd JAVA implementations are JNIs 
> https://github.com/MeteoGroup/jbrotli (ASL 2)
> https://github.com/luben/zstd-jni (3 clause BSD)



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to