Conversation
|
What about 1.16? |
I think flink-1.16 and spark-3.3 is handled in the original PR itself. |
|
nit: we could at least split it into one spark and one flink PR or one PR for each version. |
|
Thanks @chenjunjiedada @ajantha-bhat for the review.
It is already done in #4627.
OK, let me split it into two patches. |
|
Well, I thought flink 1.16 was supported recently. Never thought the change was already in it. |
|
cc @szehon-ho @pvary @kbendick who reviewed the original pr. |
|
@ConeyLiu - was it a clean backport, or we should specifically check some changes which are different from the original fix? |
|
@pvary same here, just copy, no other changes. |
|
@ConeyLiu: Thanks for the PR |
|
Thanks @pvary for merging this and thanks @chenjunjiedada @ajantha-bhat for the review. |
This PR just ported #4627 to Spark 2.4/3.1/3.2.