Clickhouse: High version is not compatible with CatBoost

[jackpgao]

• With the following config, we can use a trained catboost model to predict data.

cat catboost/models/trips_price_model.xml
<models>
    <model>
        <!-- Model type. Now catboost only. -->
        <type>catboost</type>
        <!-- Model name. -->
        <name>trip_price</name>
        <!-- Path to trained model. -->
        <path>/data1/clickhouse/catboost/models/trip_price.bin</path>
        <!-- Update interval. -->
        <lifetime>0</lifetime>
    </model>
</models>

• result

SELECT
    modelEvaluate('trip_price', trip_distance, toYear(pickup_datetime), cab_type) AS prediction,
    total_amount
FROM trips_mergetree_all
LIMIT 5

┌─────────prediction─┬─total_amount─┐
│  9.245315066134815 │          9.2 │
│ 10.368612172918901 │         10.8 │
│  9.931151401302877 │          8.8 │
│ 18.997186263989573 │         14.5 │
│   7.11047499713387 │          6.3 │
└────────────────────┴──────────────┘

5 rows in set. Elapsed: 0.009 sec.


SELECT version()

┌─version()──┐
│ 19.11.9.52 │
└────────────┘

• but the higher version can not parsed the catboost model config file.

• error.log

2020.01.08 14:24:15.691965 [ 1 ] {} <Warning> ExternalModelsLoader: /data1/clickhouse/catboost/models/trips_price_model.xml: file contains unknown node 'model', expected 'models'

SELECT version()

┌─version()────┐
│ 19.19.1.1957 │
└──────────────┘

1 rows in set. Elapsed: 0.002 sec.


SELECT
    modelEvaluate('trip_price', trip_distance, toYear(pickup_datetime), cab_type) AS prediction,
    total_amount
FROM trips_mergetree_all
LIMIT 5

Received exception from server (version 19.19.1):
Code: 36. DB::Exception: Received from 127.0.0.1:9000. DB::Exception: external model 'trip_price' not found.

• and the config in this slide: https://github.com/ClickHouse/clickhouse-presentations/blob/master/meetup10/catboost.pdf is also not right.

[JamalSale]

Confirm, have the same issue with

SELECT version()

┌─version()──┐
│ 19.17.6.36 │
└────────────┘

[KochetovNicolai]

That was the reason: https://github.com/ClickHouse/ClickHouse/commit/42692ad07b6e37fe3697c6611fcc672f358048bb
Unfortunately, I've missed the slides source, so can't fix mistake there.