[jira] [Commented] (HELIX-107) Add support to set custom objects into ZNRecord

["Kanak Biscuitwala (JIRA)" <ji...@apache.org>]

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

Kanak Biscuitwala commented on HELIX-107:
-----------------------------------------

There's two parts to this fix:

1) Clean up uses of simple fields with non-String types that have default values on failure (e.g. int, boolean, enums)
2) Let ZNRecord hold a byte array payload and a custom serializer. The API is a setter and getter passing in a generic class.

Code review: https://reviews.apache.org/r/13193/
                
> Add support to set custom objects into ZNRecord
> -----------------------------------------------
>
>                 Key: HELIX-107
>                 URL: https://issues.apache.org/jira/browse/HELIX-107
>             Project: Apache Helix
>          Issue Type: Sub-task
>          Components: helix-core, helix-webapp-admin
>            Reporter: kishore gopalakrishna
>            Assignee: Kanak Biscuitwala
>             Fix For: 0.6.2-incubating
>
>
> Currently all data has to be represented in form of simple fields, mapFields and listFields. This was done so that we can write custom serializer/deserialier so that we get good performance. We saw 30% improvement, however this has impacted code readability. 
> we need to come up with a way where we can set the application specific data using custom serializer and strongly typed object. For example, we can support znrecord.setAdditionalInfo(<T> t) where we can T can implement serialize/deserialize method.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira