Krisztián Szűcs
Krisztián Szűcs
Note that this package will switch to use columnar output formats in the future, once clickhouse extends its type support for these formats, see https://github.com/kszucs/pandahouse/issues/29 for reference.
Going to split this PR into two, then address the review comments.
@jreback @icexelloss @timothydijamco I'm not sure where are you at the migration, but please let me know if I can help you with anything. I created a `3.x.x` branch which...
Just as a sidenote: the migration may seem like a big effort, but the majority of the incompatibilities can be resolved by removing the `.op()` calls.
What I mean it may worth a try sooner than later. I'll be harder for us too to recall how to resolve the more complicated scenarios.
Definitely, that's a pretty good idea actually! Would you mind putting up a PR for it?
I like the idea, also it's kinda a prerequisite to properly serialize a bound expression. Lowering a `DatabaseTable` should be straightforward since we know its name and its ibis schema...
@cpcloud it requires another review
> > @cpcloud it requires another review > > Got it. > > What's the breaking change here? APIs in ibis.util are not guaranteed to be backwards compatible even if...