lakshmi-manasa-g opened a new pull request, #1603:
URL: https://github.com/apache/samza/pull/1603

   **Improvement**: Improve the deserialization logic used for SSP key. 
Existing logic expects two ‘.’ & 3 [fields/parts in](http://fields.in/) the 
serialized format of SSP. The logic looks for index of ‘.’ and uses substring 
to fetch the fields of SSP. this prevents the addition of a 4th field 
(KeyBucket introduced in https://github.com/apache/samza/pull/1576) to the SSP 
serialized format in a backwards compatible manner.
   
   **Changes**: New logic for deser SSP uses String.split(‘.’) which makes it 
feasible to later add the Keybucket into the SSP serde.
   **Tests**: Added one test in TestSamzaObjectMapper for serde of SSP as key
   **API changes**: None.
   **Upgrade instructions**: None
   **Usage instructions**: None
   **Backwards compatible**: yes. Does not change any existing serde format of 
the SSP as key


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to