Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error with lowercase columns #123

Open
jokin opened this issue Feb 28, 2023 · 1 comment
Open

Error with lowercase columns #123

jokin opened this issue Feb 28, 2023 · 1 comment

Comments

@jokin
Copy link

jokin commented Feb 28, 2023

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
Copy link

sheinbergon commented Jul 25, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants