fix(viz): apply uniform sorting to all nvd3 timeseries charts

[villebro]

SUMMARY

Many NVD3 timeseries viz' were featured duplicated sorting logic that was missing from others. This introduces sorting logic to the base NVD3TimeSeriesViz class removes the same from the following viz':

• NVD3TimeSeriesStackedViz (=area chart)
• HorizonViz
• RoseViz
• PartitionViz

By moving the sorting logic to the base class, this logic is now added to the following chart types that were previously missing it:

• NVD3TimeSeriesViz (=line chart)
• NVD3TimeSeriesBarViz
• NVD3CompareTimeSeriesViz

The following charts that extend the NVD3TimeSeriesViz class are unaffected, since they don't feature the sorting control:

• NVD3TimePivotViz

TEST PLAN

All eight charts that this PR affects were manually tested to ensure that they work as expected.

ADDITIONAL INFORMATION

• Has associated issue: closes [Explore][Line Viz] "Sort by" not applied when series limit is not set #13472
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[codecov]

Codecov Report

Merging #14539 (a4cfc55) into master (cee22e2) will increase coverage by 0.18%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master   #14539      +/-   ##
==========================================
+ Coverage   77.21%   77.40%   +0.18%     
==========================================
  Files         958      958              
  Lines       48351    48324      -27     
  Branches     5677     5677              
==========================================
+ Hits        37335    37403      +68     
+ Misses      10816    10721      -95     
  Partials      200      200              

Flag	Coverage Δ
hive	80.89% <50.00%> (?)
mysql	81.15% <50.00%> (+0.08%)	⬆️
postgres	81.18% <50.00%> (+0.08%)	⬆️
presto	80.88% <50.00%> (+0.08%)	⬆️
python	81.71% <50.00%> (+0.34%)	⬆️
sqlite	80.79% <50.00%> (+0.08%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
superset/viz.py	56.35% <50.00%> (+0.85%)	⬆️
superset/utils/core.py	89.00% <0.00%> (+0.12%)	⬆️
superset/connectors/sqla/models.py	90.07% <0.00%> (+0.24%)	⬆️
superset/db_engine_specs/hive.py	87.39% <0.00%> (+17.07%)	⬆️
superset/db_engines/hive.py	82.14% <0.00%> (+82.14%)	⬆️

---

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 cee22e2...a4cfc55. Read the comment docs.

[zhaoyongjie]

LGTM! Thanks ville.