Zhanghao Chen
Zhanghao Chen
@HuangZhenQiu Hi, it'll be better not to squash your commits before the review is finished, so that reviewers can easily track what has changed since the last review.
@HuangZhenQiu LGTM. @FangYongs Could you help take a final look and merge it?
@reswqa Hi, could you help take a look when you have time?
@reswqa Thanks for the review. I've rebased on latest master branch, and it has been highlighted in the doc content as follows: > Currently, only `Map`, `List` and its super...
> Thanks @X-czh, I think we only need two commits. One for the implementation and one for documentation. Thanks for the notice, @reswqa. I've rearranged the commits to have only...
@kumar-mallikarjuna Thanks for the contribution. LGTM except for two minor comments, PTAL.
Pinging @FangYongs and @reswqa. Could you help take a look when you have time?
@flinkbot run azure
@pltbkd Thanks for the insightful comment! > 1. MapSerializer does not support null key, which means it needs an enhancement like this as well. You are right, we'll need to...
After offline discussion with @reswqa, we think a better way is to create a set of new nullable `TypeInformation`s whose serializer is a simple wrap of the existing serializers with...