Feat/pg semicolon support

[Jordan-M-Young]

Implementing support for finishing postgres queries with a semicolon per this issue.

Changes include:
- replace statements to remove semicolon from query in connectorx-python/connectorx/init.py (read_sql)
- Tests to confirm that command terminating semicolons are being removed.

[Jordan-M-Young]

@wangxiaoying I think this one is ready for review. Let me know if/what changes are needed. Thanks in advance!

[wangxiaoying]

Hi @Jordan-M-Young , thanks for the PR!

I think my concern is that ';' may also appears in other parts of the query, such as string matching. For example: select s from t where s like '%abc;'. In this case, simply replace ";" with "" will change the semantic of the sql query.

I would suggest to only replace the ";" if it is the last valid character in the sql query. Such as select s from t; or select s from t;[spaces] but not select s from t where s like '%abc;'

[Jordan-M-Young]

@wangxiaoying Thanks for the suggestion. I believe my newest commit should allay that concern. If you wouldn't mind taking another look I'd appreciate it.

[wangxiaoying]

Thank you @Jordan-M-Young ! I have merged the PR.

[surister]

This looks a bit cumbersome. Could be rewritten to:

if query.endswith(';'):
    query = query[:-1]
return query

[wangxiaoying]

Thanks @surister , indeed this way can be better. Can you open a PR for this?

[surister]

You got it.