## What is the purpose of the change

This change allows to store null values with TTL and aligns the semantics of 
map state TTL get/contains methods with the map state w/o TTL.

## Brief change log

  - allow null user value in TtlValue
  - add tests for null user values in Map state
  - add NullableSerializer to wrap serializers w/o null support but where null 
values are needed in map state
  - doc notes

## Verifying this change

*(Please pick either of the following options)*

This change is a trivial rework / code cleanup without any test coverage.

*(or)*

This change is already covered by existing tests, such as *(please describe 
tests)*.

*(or)*

This change added tests and can be verified as follows:

*(example:)*
  - *Added integration tests for end-to-end deployment with large payloads 
(100MB)*
  - *Extended integration test for recovery after master (JobManager) failure*
  - *Added test that validates that TaskInfo is transferred only once across 
recoveries*
  - *Manually verified the change by running a 4 node cluser with 2 JobManagers 
and 4 TaskManagers, a stateful streaming program, and killing one JobManager 
and two TaskManagers during the execution, verifying that recovery happens 
correctly.*

## Does this pull request potentially affect one of the following parts:

  - Dependencies (does it add or upgrade a dependency): (yes / no)
  - The public API, i.e., is any changed class annotated with 
`@Public(Evolving)`: (yes / no)
  - The serializers: (yes / no / don't know)
  - The runtime per-record code paths (performance sensitive): (yes / no / 
don't know)
  - Anything that affects deployment or recovery: JobManager (and its 
components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)
  - The S3 file system connector: (yes / no / don't know)

## Documentation

  - Does this pull request introduce a new feature? (yes / no)
  - If yes, how is the feature documented? (not applicable / docs / JavaDocs / 
not documented)


[ Full content available at: https://github.com/apache/flink/pull/6707 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to