eric-maynard commented on code in PR #1862: URL: https://github.com/apache/polaris/pull/1862#discussion_r2243628751
########## plugins/spark/README.md: ########## @@ -124,3 +124,9 @@ Following describes the current functionality limitations of the Polaris Spark c 3) Rename a Delta table is not supported. 4) ALTER TABLE ... SET LOCATION is not supported for DELTA table. 5) For other non-Iceberg tables like csv, it is not supported today. + +### Hudi Support +Currently support for Hudi tables within the Polaris catalog is still under development. Review Comment: It's less that the README is confusing (as Yufei said, it's normal to merge things that are in-development within a project) and more that it's confusing why we would merge something into Polaris because we expect an upstream dependency to eventually change. Couldn't we just forego the README note and hold the Polaris change until the dependency is ready? -- 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]
