dimas-b commented on issue #4663: URL: https://github.com/apache/polaris/issues/4663#issuecomment-4672735571
AI (Codex) analysis: ``` About 38% of PolarisRESTCatalog’s implementation lines are shared with Iceberg’s RESTSessionCatalog. ``` For `BasePolarisSparkCatalog`: ``` So I’d cite about 47% meaningful exact source-line overlap, or about 63% if counting all implementation lines including Spark interface boilerplate. The ordered block similarity is much lower, around 29% best-window, so this looks more like a wrapper using the same Spark catalog API than a direct line-by-line copy of an Iceberg class. ``` My take: both `PolarisRESTCatalog` and `BasePolarisSparkCatalog` are independent Polaris classes now, "inspired by" Iceberg code, but not a full "copy". -- 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]
