Skip to content

Conversation

@mbalassi
Copy link
Contributor

Resolves the following issues:

  • Streaming sometimes used Java's SerializationUtils instead of Flink's InstantiationUtils.
  • Streaming stored the userfunctions twice in the operators, once named as userfunction, once named and typed specifically (e.g. flatMapper). With a tricky serialization we could end up with two distinct userfunctions after deserialization.

@gyfora
Copy link
Contributor

gyfora commented Apr 24, 2015

It's a shame that we need to remove additional references to make the serialization work properly, but otherwise it looks good.

Let's merge it soon

@mbalassi mbalassi force-pushed the streaming-serialize branch from 6098ee1 to e6ff93b Compare April 24, 2015 09:42
@mbalassi mbalassi force-pushed the streaming-serialize branch from e6ff93b to 046f39e Compare April 24, 2015 13:18
@asfgit asfgit merged commit 046f39e into apache:master Apr 24, 2015
@mbalassi mbalassi deleted the streaming-serialize branch April 29, 2015 14:31
bhatsachin pushed a commit to bhatsachin/flink that referenced this pull request May 5, 2015
marthavk pushed a commit to marthavk/flink that referenced this pull request Jun 9, 2015
pnowojski pushed a commit to pnowojski/flink that referenced this pull request Mar 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants