Revert "Allow bigint to be used as time column in Presto" #7670

[john-bodley]

Per #7218 (comment) including a BIGINT column type as a time column in Presto is problematic as it could represent either a Unix time in seconds or milliseconds.

The current correct way to handle Unix time columns is to set the python_date_format as either epoch or epoch_ms (depending on the encoding) which results in queries comprising the correct fidelity:

SELECT COUNT(*) AS "count"
FROM "superset"."logs"
WHERE "dttm" >= 1553644800
  AND "dttm" <= 1554249600
ORDER BY "count" DESC
LIMIT 10000;

Note #7656 plans to auto-detect the encoding of temporal columns.

to: @betodealmeida @mistercrunch

[john-bodley]

@mistercrunch this is a replacement to #7670 which was problematic after the split up of db_engine_specs.py.

[codecov-io]

Codecov Report

Merging #7678 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #7678      +/-   ##
==========================================
+ Coverage   65.71%   65.71%   +<.01%     
==========================================
  Files         459      459              
  Lines       21873    21871       -2     
  Branches     2403     2403              
==========================================
  Hits        14373    14373              
+ Misses       7379     7377       -2     
  Partials      121      121

Impacted Files	Coverage Δ
superset/db_engine_specs/presto.py	69.71% <ø> (+0.33%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 95291fa...375bc76. Read the comment docs.

[codecov-io]

Codecov Report

Merging #7678 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #7678      +/-   ##
==========================================
+ Coverage   65.71%   65.71%   +<.01%     
==========================================
  Files         459      459              
  Lines       21873    21871       -2     
  Branches     2403     2403              
==========================================
  Hits        14373    14373              
+ Misses       7379     7377       -2     
  Partials      121      121

Impacted Files	Coverage Δ
superset/db_engine_specs/presto.py	69.71% <ø> (+0.33%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 95291fa...375bc76. Read the comment docs.

[mistercrunch]

LGTM