ajantha-bhat commented on pull request #3579: URL: https://github.com/apache/iceberg/pull/3579#issuecomment-974113711
@rymurr : This looks like a right direction to me. But I have some doubts. a) How user can register their procedures without extending base catalog ? Now looks like still we need to extend catalog to gain access to these methods. b) And regarding disable of existing procedures, we have not removed look up `SparkProcedures.newBuilder(name)` in the PR, so I would assume we need to disable by adding dummy procedure for them. Is my assumption is correct ? -- 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]
