[FLINK-34078] Move InternalKeyContext classes from o.a.f.runtime.state.heap to o.a.f.runtime.state package

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.19.0
• Component/s: Runtime / State Backends
• Labels: pull-request-available
• Assignee: Jinzhong Li
• Reporter: Jinzhong Li
• Created: 15/Jan/24 05:30
• Updated: 16/Jan/24 05:50
• Resolved: 16/Jan/24 02:28

Description

Motication:

When Rocksdb statebackend throws a keyGroup check illegal exception, 
the exception stack contains the heap stateBackend scoped class, which looks so strange to user.

Proposed changes:

InternalKeyContext and InternalKeyContextImpl are commonly used by all state backends (heap/rocksdb/changelog), they should be moved from org.apache.flink.runtime.state.heap package to org.apache.flink.runtime.state package.

Compatibility:

InternalKeyContext is annotated with @Internal, so this change has no compatibility issues.

 

 

Issue Links

• links to: GitHub Pull Request #24093