shenzhu commented on pull request #24: URL: https://github.com/apache/flink-table-store/pull/24#issuecomment-1049484761
> Hey @tsreaper , thanks for your feedback! I think that's a good point I'm still new to this code base, please correct me if I didn't understand it correctly. I have two questions about that idea **Q1.** Is there an easy way to know if a manifest file has been merged before? I was checking the fields in [ManifestFileMeta](https://github.com/apache/flink-table-store/blob/master/flink-table-store-core/src/main/java/org/apache/flink/table/store/file/manifest/ManifestFileMeta.java#L38) but didn't find any indicator **Q2.** Does order of operations matter? If we plan to scan from tail to head, then we might see some files deleted first and then added. For example with the following case: ``` merged1, merged2, manifest1(+A), manifest2(+B, +C), manifest3(-B, +D), entry1(-A) ``` The merged results could be something like ``` merged1, merged2, merged3(+A), merged4(+C, +D, -A) ``` -- 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]
