weiqingy commented on PR #871:
URL: https://github.com/apache/flink-agents/pull/871#issuecomment-4897968905
Closing this in favor of #874, which supersedes it.
Per the discussion on #872, rather than land this narrow top-level `byte[]`
envelope and then generalize it, we fold everything into one binary change:
#874 switches `MemoryUpdate.value` to a Flink-native (Kryo) binary payload
inside a versioned `{serde, version, payload}` envelope, which covers the
top-level `byte[]` case this PR fixed plus nested `byte[]`, `Long` narrowing,
and user POJOs — and retires the `__flink_agents_bytes__` envelope. Nothing
here is persisted in that format, so there is no migration concern.
--
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]
