wayne-x-wang commented on PR #7793:
URL: https://github.com/apache/kyuubi/pull/7793#issuecomment-6335620477

   Thanks for the guidance. I agree that Kyuubi should use Spark’s standard 
ViewCatalog API rather than introduce a Paimon-specific adapter.
   I checked Paimon’s latest release, 2.1.0, and its current master. Both still 
use Paimon’s own SupportView interface, and the published Spark connectors 
currently go up to Spark 4.1. My original reproduction uses Spark 3.5 and 
Paimon 2.0, so switching Kyuubi to the standard API would not resolve that 
case. Upgrading Spark alone would also be insufficient until Paimon adopts the 
standard interface.
   Would it make sense to rework this PR into generic GetTables support for 
Spark 4.2’s ViewCatalog, removing the Paimon-specific adapter and test setup? I 
would update the title, description, and tests to reflect that scope, and leave 
#7772 unresolved pending Paimon’s migration. Alternatively, would you prefer to 
defer this work until a supported connector provides a concrete integration 
case?


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to