Deprecate tableau extra (apache#13595)

There was no separate provider for `tableau` and it's dependencies have been already incorporated in ``salesforce`` one therefore this change deprecates it in favour of ``salesforce``.
DeepHiveMind · Jan 10, 2021 · ef0f5cd · ef0f5cd
1 parent 46edea3
commit ef0f5cd
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/docs/apache-airflow/extra-packages-ref.rst b/docs/apache-airflow/extra-packages-ref.rst
@@ -203,8 +203,6 @@ can be reported by some tools (even if it is harmless).
 +---------------------+-----------------------------------------------------+------------------------------------------------------------------------------------+-----------+
 | statsd              | ``pip install 'apache-airflow[statsd]'``            | Needed by StatsD metrics                                                           |           |
 +---------------------+-----------------------------------------------------+------------------------------------------------------------------------------------+-----------+
-| tableau             | ``pip install 'apache-airflow[tableau]'``           | Tableau visualization integration                                                  |           |
-+---------------------+-----------------------------------------------------+------------------------------------------------------------------------------------+-----------+
 | virtualenv          | ``pip install 'apache-airflow[virtualenv]'``        | Running python tasks in local virtualenv                                           |           |
 +---------------------+-----------------------------------------------------+------------------------------------------------------------------------------------+-----------+
 
@@ -250,6 +248,8 @@ all replaced by new extras, which have naming consistent with the names of provi
 The ``crypto`` extra is not needed any more, because all crypto dependencies are part of airflow package,
 so there is no replacement for ``crypto`` extra.
 
+The ``tableau`` extra dependencies have been included in the ``salesforce`` extra.
+
 +---------------------+-----------------------------+
 | Deprecated extra    | Extra to be used instead    |
 +=====================+=============================+
@@ -285,6 +285,8 @@ so there is no replacement for ``crypto`` extra.
 +---------------------+-----------------------------+
 | spark               | apache.spark                |
 +---------------------+-----------------------------+
+| tableau             | salesforce                  |
++---------------------+-----------------------------+
 | webhdfs             | apache.webhdfs              |
 +---------------------+-----------------------------+
 | winrm               | microsoft.winrm             |

diff --git a/setup.py b/setup.py
@@ -449,9 +449,6 @@ def get_sphinx_theme_version() -> str:
 statsd = [
     'statsd>=3.3.0, <4.0',
 ]
-tableau = [
-    'tableauserverclient~=0.12',
-]
 telegram = [
     'python-telegram-bot==13.0',
 ]
@@ -626,7 +623,6 @@ def get_sphinx_theme_version() -> str:
     'rabbitmq': rabbitmq,
     'sentry': sentry,
     'statsd': statsd,
-    'tableau': tableau,
     'virtualenv': virtualenv,
 }
 
@@ -666,6 +662,7 @@ def add_extras_for_all_providers() -> None:
     'qds': 'qubole',
     's3': 'amazon',
     'spark': 'apache.spark',
+    'tableau': 'salesforce',
     'webhdfs': 'apache.webhdfs',
     'winrm': 'microsoft.winrm',
 }