Airblader opened a new pull request #16631:
URL: https://github.com/apache/flink/pull/16631


   ## What is the purpose of the change
   
   This exposes `JoinedRowData` as public API. When implementing Table API 
connectors, the need to join `RowData` arises naturally from having to append 
metadata to the produced physical row. We should therefore make it easy for 
implementors to do so without relying on internal APIs.
   
   This PR makes the class public, updates its documentation and adds some 
basic tests for it.
   
   The PR is split into two commits: the actual change to `JoinedRowData`, and 
a second commit moving it into the same package as `RowData`, `GenericRowData`, 
…
   
   ## Verifying this change
   
   This change is already covered by existing tests. However, 
`JoinedRowDataTest` has also been added.
   
   ## Does this pull request potentially affect one of the following parts:
   
     - Dependencies (does it add or upgrade a dependency): no
     - The public API, i.e., is any changed class annotated with 
`@Public(Evolving)`: yes
     - The serializers: no
     - The runtime per-record code paths (performance sensitive): "yes", but no 
actual runtime behavior has been changed
     - Anything that affects deployment or recovery: JobManager (and its 
components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
     - The S3 file system connector: no
   
   ## Documentation
   
     - Does this pull request introduce a new feature? no
     - If yes, how is the feature documented? JavaDocs
   


-- 
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]


Reply via email to