Idea to refactor number code

[clach04]

Related to:

1. comment Add preliminary Ingres/Vector/Actian Avalanche support #36 (comment)
2. change / commit 4f8ad30

Idea / Proposal / Question

agate-sql/agatesql/table.py

Line 215 in d2bd282

if isinstance(column.data_type, agate.Number) and dialect in ('ingres', 'mssql', 'mysql', 'oracle'):

Change the include list check to an exclude list. Looking at the code, sounds like PostgreSQL and SQLite could be an exclude list, and maybe easier to maintain.

            if isinstance(column.data_type, agate.Number) and dialect not in ('postgresql', 'sqlite'):

[jpmckinney]

As far as maintenance goes, it’s the same to me whether we include or exclude. What’s important is to document the precision ranges, so that we set a valid default.