{"action":"created","issue":{"url":"https://api.github.com/repos/twitter/chill/issues/216","labels_url":"https://api.github.com/repos/twitter/chill/issues/216/labels{/name}","comments_url":"https://api.github.com/repos/twitter/chill/issues/216/comments","events_url":"https://api.github.com/repos/twitter/chill/issues/216/events","html_url":"https://github.com/twitter/chill/pull/216","id":55969310,"number":216,"title":"Add build instructions to readme and make InjectiveSerializer serializable","user":{"login":"rmetzger","id":89049,"avatar_url":"https://avatars.githubusercontent.com/u/89049?v=3","gravatar_id":"","url":"https://api.github.com/users/rmetzger","html_url":"https://github.com/rmetzger","followers_url":"https://api.github.com/users/rmetzger/followers","following_url":"https://api.github.com/users/rmetzger/following{/other_user}","gists_url":"https://api.github.com/users/rmetzger/gists{/gist_id}","starred_url":"https://api.github.com/users/rmetzger/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rmetzger/subscriptions","organizations_url":"https://api.github.com/users/rmetzger/orgs","repos_url":"https://api.github.com/users/rmetzger/repos","events_url":"https://api.github.com/users/rmetzger/events{/privacy}","received_events_url":"https://api.github.com/users/rmetzger/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-01-29T22:48:58Z","updated_at":"2015-01-29T23:24:28Z","closed_at":"2015-01-29T23:22:12Z","pull_request":{"url":"https://api.github.com/repos/twitter/chill/pulls/216","html_url":"https://github.com/twitter/chill/pull/216","diff_url":"https://github.com/twitter/chill/pull/216.diff","patch_url":"https://github.com/twitter/chill/pull/216.patch"},"body":"Hi,\r\n\r\nApache Flink recently started using chill and Kryo as a fallback serializer. We really like the work done here.\r\nSince we are sending the serializers around in the cluster, they need to be Serializable. I found that the `InjectiveSerializer` is not Serializable, so I made it implement the interface.\r\n\r\nI also added some build instructions to the readme, because it took me some time to research the sbt commands.\r\n\r\nPlease let me know if you want something in this pull request to be solved differently."},"comment":{"url":"https://api.github.com/repos/twitter/chill/issues/comments/72125304","html_url":"https://github.com/twitter/chill/pull/216#issuecomment-72125304","issue_url":"https://api.github.com/repos/twitter/chill/issues/216","id":72125304,"user":{"login":"johnynek","id":67958,"avatar_url":"https://avatars.githubusercontent.com/u/67958?v=3","gravatar_id":"","url":"https://api.github.com/users/johnynek","html_url":"https://github.com/johnynek","followers_url":"https://api.github.com/users/johnynek/followers","following_url":"https://api.github.com/users/johnynek/following{/other_user}","gists_url":"https://api.github.com/users/johnynek/gists{/gist_id}","starred_url":"https://api.github.com/users/johnynek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/johnynek/subscriptions","organizations_url":"https://api.github.com/users/johnynek/orgs","repos_url":"https://api.github.com/users/johnynek/repos","events_url":"https://api.github.com/users/johnynek/events{/privacy}","received_events_url":"https://api.github.com/users/johnynek/received_events","type":"User","site_admin":false},"created_at":"2015-01-29T23:24:28Z","updated_at":"2015-01-29T23:24:28Z","body":"Second what Ian said about us generally avoiding serializing serializers. Since Kryo is mutable, we generally use `KryoInstantiator` instances to create new ones.  Also there is `KryoPool` so you can safely share instances without allocating each time you need one.\r\n\r\nConfiguredInstantiator is one way we solve this problem across a lot of use cases."}}