Add with_properties and interpolated_properties

[surister]

Summary of the changes / Why this is an improvement

This PR allows for extracting properties defined by [ WITH ( parameter [= value] [, ... ] ) ] statements.

For example:

from cratedb_sqlparse import sqlparse

stmt = sqlparse("""
    CREATE TABLE doc.tbl12 (A TEXT) WITH (
      "allocation.max_retries" = 5,
      "blocks.metadata" = false
    );
""")[0]

print(stmt.metadata)
# Metadata(schema='doc', table_name='tbl12', interpolated_properties={}, with_properties={'allocation.max_retries': '5', 'blocks.metadata': 'false'})

It also handles the special case of what I call, interpolated values and #31 if you have a better name recommendation, please go ahead :P

Checklist

• Link to issue this PR refers to (if applicable):
• CLA is signed

[matriv]

I don't get what's the value of splitting those into an extra key in metadata.
Could you please explain what is the thought behind this?

(If we keep it, for the name I think Query parameterized properties sounds a bit better)

[surister]

Just conveniency, so users don't need to constantly filter properties to get those that are parametrized, without this I see myself (as an user) extending the API to filter this parameters.

I like parameterized properties. 👍

[amotl]

You might be interested in this utility methods, and their corresponding improvements?

• Dialect: Fix get_table_names() reflection method sqlalchemy-cratedb#10
• Use sqlalchemy's 'quote' function to quote table names and fix table quoting in cfr exporter cratedb-toolkit#172
• Utility function for properly quoting table names, full-qualified or not sqlalchemy-cratedb#69

[matriv]

Suggested change

	In this case, both `blocks.metadata` will be in `with_properties` and `interpolated_properties`.
	In this case, `blocks.metadata` will be in `with_properties` and `interpolated_properties` as well.

[matriv]

Shouldn't we also assert the parameterized_properties here?

[surister]

👌 Not only that but we can just do

    expected = {'key': 'val', 'key2': '2'}

    # Has all the keys.
    assert stmt.metadata.with_properties == expected