You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@orc.apache.org by "Owen O'Malley (JIRA)" <ji...@apache.org> on 2019/05/29 04:50:00 UTC
[jira] [Updated] (ORC-510) Cleanup API for StreamOptions and
CompressionCodec.Options
[ https://issues.apache.org/jira/browse/ORC-510?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Owen O'Malley updated ORC-510:
------------------------------
Description:
The writer's stream options should be cleaned up. There are a couple of related items:
* We end up change the IV's for streams and we need to notify the stream when we do so, therefore it is better to make an explicit API to change them.
* CryptoUtils should match with functions to modify an IV rather than forcing an allocation.
* CompressionCodec.Options only needs to be copied when we have tuning options.
* CompressionCodec would be nice to have as Closeable where that returns it to the pool.
was:Currently, we always need to copy the CompressionCodec.Options even for codecs that don't have options.
Summary: Cleanup API for StreamOptions and CompressionCodec.Options (was: Simplify API for CompressionCodec.Options)
> Cleanup API for StreamOptions and CompressionCodec.Options
> ----------------------------------------------------------
>
> Key: ORC-510
> URL: https://issues.apache.org/jira/browse/ORC-510
> Project: ORC
> Issue Type: Sub-task
> Reporter: Owen O'Malley
> Assignee: Owen O'Malley
> Priority: Major
>
> The writer's stream options should be cleaned up. There are a couple of related items:
> * We end up change the IV's for streams and we need to notify the stream when we do so, therefore it is better to make an explicit API to change them.
> * CryptoUtils should match with functions to modify an IV rather than forcing an allocation.
> * CompressionCodec.Options only needs to be copied when we have tuning options.
> * CompressionCodec would be nice to have as Closeable where that returns it to the pool.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)