[SPARK-43041] Restore constructors of exceptions for compatibility in connector API

• Type: Bug
• Status: Resolved
• Priority: Blocker
• Resolution: Fixed
• Affects Version/s: 3.4.0
• Fix Version/s: 3.4.0
• Component/s: Spark Core
• Assignee: Anton Okolnychyi
• Reporter: Xinrong Meng
• Created: 05/Apr/23 19:03
• Updated: 06/Apr/23 05:26
• Resolved: 06/Apr/23 05:11

Description

Thanks aokolnychyi for raising the issue as shown below:

I have a question about changes to exceptions used in the public connector API, such as NoSuchTableException and TableAlreadyExistsException.

I consider those as part of the public Catalog API (TableCatalog uses them in method definitions). However, it looks like PR #37887 has changed them in an incompatible way. Old constructors accepting Identifier objects got removed. The only way to construct such exceptions is either by passing database and table strings or Scala Seq. Shall we add back old constructors to avoid breaking connectors?

We should restore constructors of those exceptions to preserve the compatibility in connector API.

Issue Links

• links to: [Github] Pull Request #40679 (aokolnychyi)