panbingkun commented on PR #47364: URL: https://github.com/apache/spark/pull/47364#issuecomment-2337385675
> @panbingkun Could you clarify, please, why do you implement this new command using V1 DataSource but not V2? This just require unnecessary effort for porting it on DataSource V2 API. I'm not sure if I need to add an interface like `CollationCatalog`, Of course, if we don't add such an interface for now, it seems possible to have the method exist in `V2SessionCatalog` first. However, based on my understanding, it seems more reasonable to add the interface `CollationCatalog`. Is that correct? @MaxGekk -- 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]
