crate / commons-codec

Data decoding, encoding, conversion, and translation utilities.
https://commons-codec.readthedocs.io
GNU Lesser General Public License v3.0
2 stars 2 forks source link

Replace poor man's relation name quoting with implementation from `sqlalchemy-cratedb` #38

Closed amotl closed 2 months ago

amotl commented 2 months ago

About

When needed, use quote_relation_name from sqlalchemy-cratedb package. Otherwise, don't quote at all, and push this responsibility to the caller. I think it is better doing it this way, than doing it wrong.

Instead of doing it insufficiently, let's pay the price on pulling in the sqlalchemy-cratedb package as a dependency, to be able to use the canonical implementation quote_relation_name without further ado.