Lin Liu

@yihua, @jonvex , "hoodie.schema.add.field.ids" seems not a valid config. Do you know if we should keep this PR?

@yihua @codope @danny0405

Will clean the failures.

This is not a correct direction for cdc demo.

@jonvex , the tests are failing.

> Did we chase down every reader (base files, log files, iterator) part or the new file group and ensured this is the only gap ? @jonvex @linliu-code I haven't....

Overall it looks solid to me. Only concern is that this code is very critical and complex, do we have enough tests to ensure the correctness?

I think this also failed the master. Why just the 0.X branch? Meanwhile, when is this bug introduced? I start to notice the OOM problem recently.

Any updates?

> Hi, @linliu-code do we have a clear tutorial on how to use the new mergers, should we add a new strategy id for each merger class or just use...