Constantine Peresypkin

I totally agree with your assessment. It's C++, any time you step into UB all bets are off... But have no choice right now. I'm cleaning up the process, but...

Overall I like it a lot. It's the only tool that does what I need.

@kitaisreal I'm not sure how to pass params into function so it doesn't require escaping too much Do you have an idea for a good syntax ?

@kitaisreal in each script we need to map position to key and it works essentially in UDF only, not in UDTF? Dunno if it's good enough. Env vars is a...

> SETTINGS a = b ENVIRONMENT c = d Yeah, I've also considered that one. If it's better, I can do it. > do nothing, let all the variables passed...

> Networking is needed for some UDFs Here I mean that the namespace for networking will not change, as there is no point to mess with it. Will be equivalent...

@alexey-milovidov @kitaisreal I'm not sure where in config should I put the `allowed_env_variables` expression? For UDF and UDict it's anyway configured from config only, but there is no specific config...

@alexey-milovidov I still cannot get the exact syntax, for example if the table function looks like: ```sql SELECT * FROM executable('my_model.py predict', 'ArrowStream', 'prediction Float32', (SELECT * FROM table), ENVIRONMENT...

@kitaisreal @alexey-milovidov I'm hitting a wall here I've tried to create a good parser for _partial settings_ by inheriting from the current "SetQuery" parser ```C++ class ASTSettings : public ASTSetQuery...

Ah, found it! `ASTPtr clone() const override { return std::make_shared(*this); }` clone always called the parent... Indeed the tree gets reassembled through cloning...