Skip to content

Commit

Permalink
[Bug fix] Divide by 1000.000 in epoch_ms_to_dttm() to not lose precis…
Browse files Browse the repository at this point in the history
…ion in Presto (apache#5211)

* Fix how the annotation layer interpretes the timestamp string without timezone info; use it as UTC

* [Bug fix] Fixed/Refactored annotation layer code so that non-timeseries annotations are applied based on the updated chart object after adding all data

* [Bug fix] Fixed/Refactored annotation layer code so that non-timeseries annotations are applied based on the updated chart object after adding all data

* Fixed indentation

* Fix the key string value in case series.key is a string

* Fix the key string value in case series.key is a string

* [Bug fix] Divide by 1000.000 in epoch_ms_to_dttm() to not lose precision in Presto

* [Bug fix] Divide by 1000.000 in epoch_ms_to_dttm() to not lose precision in Presto
  • Loading branch information
EvelynTurner authored and mistercrunch committed Jul 4, 2018
1 parent da5a7de commit 373d8f0
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion superset/db_engine_specs.py
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ def epoch_to_dttm(cls):

@classmethod
def epoch_ms_to_dttm(cls):
return cls.epoch_to_dttm().replace('{col}', '({col}/1000.0)')
return cls.epoch_to_dttm().replace('{col}', '({col}/1000.000)')

@classmethod
def get_datatype(cls, type_code):
Expand Down

0 comments on commit 373d8f0

Please sign in to comment.