Sanil15 opened a new pull request #1486: URL: https://github.com/apache/samza/pull/1486
***Feature/Issue:*** BaseKeyValueStorageEngineFactory assumes non-null serdes for k,v of stores using it and applies check/ ContainerStorageManager also applies redundant check to verify the same, however, this might not hold true for some StorageEngine use cases, since some stores may elect to not expose serdes but just use Java Objects as K, V ***Changes:*** - Remove the redundant check from ContainerStorageManager that assumes each store should have non-null serdes - Add docs to BaseKeyValueStorageEngineFactory on serde assumption - Add more docs on assumptions for serdes and existence between wrappedStore & rawStore in KeyValueStorageEngine API ***API Changes: None*** ***Upgrade instructions: None*** -- 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. For queries about this service, please contact Infrastructure at: [email protected]
