theoryxu
theoryxu
> @theoryxu it would be nice if you could add a UT for this feature, to demonstrate the behavior differences with this feature enabled/disabled. @pan3793 Thank you for your review....
@pan3793 Is anyone working on this? If not, I can submit a PR for it recently and then discuss it in more detail.
@pan3793 Hi bro, I have submitted a PR. Could you review it? Thanks.
@FANNG1 @jerryshao please help to review, Thanks.
> thanks for proposing the PR, some quick thought: > > 1. how should we present the `prefix`, name or id? how other vendors like tabular organize it? > 2....
> could you add document to `docs/iceberg-rest-service.md` ? ok, I have added the content to it.
@FANNG1 could you help review again?
@FANNG1 All checks have passed. Could you help review it again? Thanks
> LGTM except few comments, it's nearly ready to merge, @jerryshao do you have time to review again? All remaining flaws have been solved. Thanks for reviewing it.
@FANNG1 please help to review, Thanks.