You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/01/11 15:47:34 UTC

[GitHub] [airflow] potiuk opened a new issue #13614: Separate out tableau provider

potiuk opened a new issue #13614:
URL: https://github.com/apache/airflow/issues/13614


   **Description**
   
   Tableau provider is currenly embedded into salesforce, but it should be removed to a separate 'tableau' provider.. This should lead to:
   
   * 1.0.1 version of the salesforce provider 
   * 1.0.0 version of the tableau provider
   
   Seems that salesforce provider might be backwards-compatible with deprecation warnings:
   
   * The classes from the salesforce provider should be moved to Tableau
   * The old classes in salesforce provider should (likely?) get a deprecation warning and import the providers from tableau
   * The salesforce provider should have a  "hard" dependency on "tableau" provider  (no extra needed)
   
   **Use case / motivation**
   
   Dicussion [here][(https://lists.apache.org/thread.html/rb1828485df78df7649000ff586a749ab685635f606b394386e1be808%40%3Cdev.airflow.apache.org%3E) has shown that the tableau provider is really independent from salesforce and we have already similar examples (slack for example). Also it makes sense not to change provider structure when ownership of a service changes:
   
   **Related Issues**
   
   #13595


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] jyotidhiman0610 commented on issue #13614: Separate out tableau provider from salesforce

Posted by GitBox <gi...@apache.org>.
jyotidhiman0610 commented on issue #13614:
URL: https://github.com/apache/airflow/issues/13614#issuecomment-770614131


   @anushkrishnav I have already started on it, I am waiting for #13767 to proceed further :) 
   
    


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] jyotidhiman0610 commented on issue #13614: Separate out tableau provider from salesforce

Posted by GitBox <gi...@apache.org>.
jyotidhiman0610 commented on issue #13614:
URL: https://github.com/apache/airflow/issues/13614#issuecomment-765507008


   @potiuk can i take this up?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] anushkrishnav commented on issue #13614: Separate out tableau provider from salesforce

Posted by GitBox <gi...@apache.org>.
anushkrishnav commented on issue #13614:
URL: https://github.com/apache/airflow/issues/13614#issuecomment-770799392


   Okayy :)


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] kaxil closed issue #13614: Separate out tableau provider from salesforce

Posted by GitBox <gi...@apache.org>.
kaxil closed issue #13614:
URL: https://github.com/apache/airflow/issues/13614


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] potiuk commented on issue #13614: Separate out tableau provider from salesforce

Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #13614:
URL: https://github.com/apache/airflow/issues/13614#issuecomment-765563519


   Surely! Just be aware we are changing the documentation generation now to accomodate SEMVER versioning for providers #13767 so there will be slight changes in the files present in the "providers" folder (no README, CHANGELOG.rst for example). so you might want to wait a bit (Monday next week I hope) until that one is merged.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] anushkrishnav commented on issue #13614: Separate out tableau provider from salesforce

Posted by GitBox <gi...@apache.org>.
anushkrishnav commented on issue #13614:
URL: https://github.com/apache/airflow/issues/13614#issuecomment-770609345


   Can I work on this ?
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org