You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@nemo.apache.org by GitBox <gi...@apache.org> on 2018/06/18 08:57:57 UTC

[GitHub] sanha opened a new pull request #48: [NEMO-72] Instance-based Encoder/Decoder interface

sanha opened a new pull request #48: [NEMO-72] Instance-based Encoder/Decoder interface
URL: https://github.com/apache/incubator-nemo/pull/48
 
 
   JIRA: [NEMO-72: Instance-based Encoder/Decoder interface](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-72)
   
   **Major changes:**
   - Modify the interface of `Coder` from static fashion to instance-based fashion 
     (to avoid to generate instance-based coder such as Spark serializer for every encoding).
   - Split `Coder` into `Encoder` and `Decoder`
     (to support some use cases to separate the encoding and decoding like in [#32](https://issues.apache.org/jira/browse/NEMO-32)).
   - Split `StreamChainer` into `EncodeStreamChainer` and `DecodeStreamChainer`.
   
   **Minor changes to note:**
   - Split `CoderProperty` into `EncoderProperty` and `DecoderProperty`
   - Split `CompressionProperty` into `CompressionProperty` and `DecompressionProperty`
   
   **Tests for the changes:**
   - Existing unit tests for `Coder`, `CoderProperty`, `Compression`, `CompressionProperty`, and other data-plain side tests cover this change.
   - Existing integration tests also cover this change.
   
   **Other comments:**
   - N/A.
   
   resolves [NEMO-72](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-72)
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services