Fixed ReplacingMergeTree EngineSpec parsing: is_deleted column presence caused error

[AlexTheKing]

Summary

Column for marking rows as deleted (is_deleted column) of ReplacingMergeTree was not anyhow handled during EngineSpec parsing resulting in runtime exception like below:

java.lang.IllegalArgumentException: Expect list size 0 or 1, but got 2
at xenon.clickhouse.parse.ParseUtils$.seqToOption(ParseUtils.scala:32)
at xenon.clickhouse.parse.AstVisitor.visitEngineClause(AstVisitor.scala:78)
at xenon.clickhouse.parse.SQLParser.$anonfun$parseEngineClause$1(SQLParser.scala:30)
at xenon.clickhouse.parse.SQLParser.parse(SQLParser.scala:49)
at xenon.clickhouse.parse.SQLParser.parseEngineClause(SQLParser.scala:30)
at xenon.clickhouse.spec.TableEngineUtils$.resolveTableEngine(TableEngineUtils.scala:24)
at xenon.clickhouse.ClickHouseCatalog.loadTable(ClickHouseCatalog.scala:133)
at xenon.clickhouse.ClickHouseCatalog.loadTable(ClickHouseCatalog.scala:36)
at org.apache.spark.sql.DataFrameWriter.saveAsTable(DataFrameWriter.scala:583)
at org.apache.spark.sql.DataFrameWriter.saveAsTable(DataFrameWriter.scala:566)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at py4j.reflection.MethodInvoker.invoke(MethodInvoker.java:244)
at py4j.reflection.ReflectionEngine.invoke(ReflectionEngine.java:357)
at py4j.Gateway.invoke(Gateway.java:282)
at py4j.commands.AbstractCommand.invokeMethod(AbstractCommand.java:132)
at py4j.commands.CallCommand.execute(CallCommand.java:79)
at py4j.ClientServerConnection.waitForCommands(ClientServerConnection.java:182)
at py4j.ClientServerConnection.run(ClientServerConnection.java:106)
at java.lang.Thread.run(Thread.java:748)

Checklist

Delete items not relevant to your PR:

• Unit tests covering the common scenario was added

[CLAassistant]

All committers have signed the CLA.

[AlexTheKing]

I'm not sure whether I have something to do with the ClickHouse Cloud tests :)

[pan3793]

@AlexTheKing the cloud test failure is irrelevant.

instead of dropping the is_deleted column info directly, I'm wondering if we can reserve such info in ReplacingMergeTreeEngineSpec, then we can use such info to filter/process the deleted rows on the spark side later if wanted.

[AlexTheKing]

I can add changes to preserve it in ReplacingMergeTreeEngineSpec if needed

[AlexTheKing]

Added one more commit: now preserving is_deleted column in spec, also added the same for ReplicatedReplacingMergeTree (it had the same issue)

[AlexTheKing]

I fixed test & style, should like OK now

[mshustov]

@mzitnik PTAL

[mzitnik]

Checking Cloud strange behavior, currently will merge PR