search

bryanyang0528 / ksql-python

A python wrapper for the KSQL REST API.
MIT License
159 stars 64 forks source link

Error with lowercase columns #123

Open jokin opened 1 year ago

jokin commented 1 year ago

the regex to parse the column names expect that they are all uppercase, but some queries and columns may be in lowercase even if it's not the default in ksqldb

regex = r"(?<!\<)`(?P<name>[A-Z_]+)` (?P<type>[A-z]+)[\<, \"](?!\>)"
sheinbergon commented 4 months ago

Hello @jokin

I've forked the code, modernized it (replaced hyper with HTTPX, bumped all dependencies, updated to Python 3.9 as the minimal version, and fixed all the tests to behave with latest KSQL DB versions (0.29.0). Could you check if it's still present in my version?

https://github.com/sheinbergon/ksql-python-ng

I'll be making a PYPI release soon enough. In the meanwhile, you can test it by running

pipx install git+https://github.com/sheinbergon/ksql-python-ng.git --include-deps or

pip install git+https://github.com/sheinbergon/ksql-python-ng.git