fix: Presto column_type_mappings time and timestamp

[rmgpinto]

SUMMARY

Presto has incorrect order of column type mappings.
timestamp is overriden with time due to these lines:

(re.compile(r"^time.*", re.IGNORECASE), types.Time()),
(re.compile(r"^timestamp.*", re.IGNORECASE), types.TIMESTAMP())

causing incorrect query statement to be passed from superset to presto.

TEST PLAN

Proposed solutions swaps those lines.

ADDITIONAL INFORMATION

• Has associated issue: Presto incorrect order of column type mappings #12860
• Changes UI
• Requires DB Migration.
• Confirm DB Migration upgrade and downgrade tested.
• Introduces new feature or API
• Removes existing feature or API

[villebro]

Thanks for the fix! Would you mind adding a unit test for this?

superset/tests/db_engine_specs/presto_tests.py

Lines 537 to 558 in 9a7fba8

	def test_get_sqla_column_type(self):
	sqla_type = PrestoEngineSpec.get_sqla_column_type("varchar(255)")
	assert isinstance(sqla_type, types.VARCHAR)
	assert sqla_type.length == 255
	sqla_type = PrestoEngineSpec.get_sqla_column_type("varchar")
	assert isinstance(sqla_type, types.String)
	assert sqla_type.length is None
	sqla_type = PrestoEngineSpec.get_sqla_column_type("char(10)")
	assert isinstance(sqla_type, types.CHAR)
	assert sqla_type.length == 10
	sqla_type = PrestoEngineSpec.get_sqla_column_type("char")
	assert isinstance(sqla_type, types.CHAR)
	assert sqla_type.length is None
	sqla_type = PrestoEngineSpec.get_sqla_column_type("integer")
	assert isinstance(sqla_type, types.Integer)
	sqla_type = PrestoEngineSpec.get_sqla_column_type(None)
	assert sqla_type is None