Although we supported legacy interfaces, we still use new interfaces to implement the deprecated interfaces.

Although we supported legacy interfaces, we still use new interfaces to implement the deprecated interfaces.

Sorry, I couldn't get your point :(

One possible way is that we implement this method in different Spark version's module, not in common module to avoid this issue?

One possible way is that we implement this method in different Spark version's module, not in common module to avoid this issue?

yes, but this introduces redundant code which is located in different spark version's module. And there seems no bad effect to using the old interface.

Although we supported legacy interfaces, we still use new interfaces to implement the deprecated interfaces.

Sorry, I couldn't get your point :(

We should have all of these methods at same time as follows.

createTable(
      Identifier ident, StructType schema, Transform[] partitions, Map<String, String> properties);

  public Table createTable(
      Identifier ident, Column[] columns, Transform[] transforms, Map<String, String> properties)

One possible way is that we implement this method in different Spark version's module, not in common module to avoid this issue?

yes, but this introduces redundant code which is located in different spark version's module. And there seems no bad effect to using the old interface.

No, the basic logic is the same and can be reused in different modules, you can define a common, version independent method.

the new interface and Column type was introduced in Spark 3.4 which couldn't be reused in older versions.

the new interface and Column type was introduced in Spark 3.4 which couldn't be reused in older versions.

the new interface and Column type was introduced in Spark 3.4 which couldn't be reused in older versions.

Maybe you needn't to reuse, but we should have the similar interface. Maybe you can use object as the type. WDYT?

let's delay this until after spark multi version design is finished. cc @jerryshao @qqqttt123