Fix long URL for from_clickhouse

kszucs / pandahouse

Pandas interface for Clickhouse database

MIT License

229 stars 70 forks source link

Fix long URL for from_clickhouse #11

Closed JLeech closed 5 years ago

JLeech commented 6 years ago

i have a case with very long "in" condition in request. default url length is 16kb, so, according to clickhouse documentation about http interface, it is possible to move query to POST body. I coded this functionality(15kb or more check, just to be sure) and made test for this case.

Welcome for comments

kszucs commented 5 years ago

@JLeech sorry for the late response, it looks good to me however has formatting issues. Could You please run flake8 on the changes?