iting0321 commented on PR #4705: URL: https://github.com/apache/polaris/pull/4705#issuecomment-4692773701
> Thanks for this @iting0321! I think we are trying to, however, do too much in this PR itself. Can we remove the Persistence-related models, as we may need a bit more time to close consensus on those bits? I understand that there will be no callers of these models as a result, but we will still need this in case of both persistence and passthrough models regardless. Given that nothing in here would be considered a "public interface" IMO, we should be ok to change it later down the line, if needed. I just removed the Persistence-related models (including ingest model). This PR now only keeps the neutral lineage query/response models, the `LineageService` query boundary, runtime config/service wiring,the disabled-by-default `ENABLE_LINEAGE` guard, and the related tests/docs. I also updated the PR description to explain what is out of scope. Thanks! -- 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]
