WHERE condition parser

[lmangani]

Hey @zacMode 👋 Just discovered this extension and its super useful! I'm not sure if this is even compatible in intention with your roadmap (absolutely no offense if not) but were about to expose a similarly simple where parser in our chsql quackscience community extension but now it seems more appropriate to perhaps have this as part of this dedicated one, so here's a little PR + Tests extension just for discussion.

D SELECT * FROM parse_where('SELECT * FROM MyTable WHERE time > 1 AND time < 100');
┌────────────────┬────────────┬─────────┐
│   condition    │ table_name │ context │
│    varchar     │  varchar   │ varchar │
├────────────────┼────────────┼─────────┤
│ ("time" > 1)   │ MyTable    │ WHERE   │
│ ("time" < 100) │ MyTable    │ WHERE   │
└────────────────┴────────────┴─────────┘

D SELECT * FROM parse_where_detailed('SELECT * FROM MyTable WHERE time > 1 AND time < 100');
┌─────────────┬───────────────┬─────────┬────────────┬─────────┐
│ column_name │ operator_type │  value  │ table_name │ context │
│   varchar   │    varchar    │ varchar │  varchar   │ varchar │
├─────────────┼───────────────┼─────────┼────────────┼─────────┤
│ time        │ >             │ 1       │ MyTable    │ WHERE   │
│ time        │ <             │ 100     │ MyTable    │ WHERE   │
└─────────────┴───────────────┴─────────┴────────────┴─────────┘

D SELECT * FROM parse_where('SELECT * FROM MyTable WHERE time BETWEEN 1 AND 100');
┌────────────────────────────┬────────────┬─────────┐
│         condition          │ table_name │ context │
│          varchar           │  varchar   │ varchar │
├────────────────────────────┼────────────┼─────────┤
│ ("time" BETWEEN 1 AND 100) │ MyTable    │ WHERE   │
└────────────────────────────┴────────────┴─────────┘

D SELECT * FROM parse_where_detailed('SELECT * FROM MyTable WHERE time BETWEEN 1 AND 100');
┌─────────────┬───────────────┬─────────┬────────────┬─────────┐
│ column_name │ operator_type │  value  │ table_name │ context │
│   varchar   │    varchar    │ varchar │  varchar   │ varchar │
├─────────────┼───────────────┼─────────┼────────────┼─────────┤
│ time        │ >=            │ 1       │ MyTable    │ WHERE   │
│ time        │ <=            │ 100     │ MyTable    │ WHERE   │
└─────────────┴───────────────┴─────────┴────────────┴─────────┘

[zacMode]

Hi @lmangani! Thanks for the contribution, this looks really cool!

I just kicked off the tests, assuming those all pass I'll get this merged

[lmangani]

The changes build and pass locally using v1.3.0 as baseline, I'll check out what's wrong with the builder/action

EDIT: I see the issue is simply with the "next" builder - changes related to 1.4.0 in main which affect the existing code as well. The v1.3.0 build should pass smoothly.

[lmangani]

@zacMode any thoughts or change requests?

[zacMode]

Apologies for the delay @lmangani ! I was hoping to have time this weekend to look into the "next build" stuff (at least to understand what it'll take to fix it), but didn't get to it.

I'll go ahead and merge and fix at a later time.

Thanks for your patience (and contribution)!

[lmangani]

Thanks @zacMode for accepting the PR! I'm looking forward to using the next extension release in our workflows!