Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Arango hook #17778

Closed
alexInhert opened this issue Aug 23, 2021 · 20 comments · Fixed by #22548
Closed

Add Arango hook #17778

alexInhert opened this issue Aug 23, 2021 · 20 comments · Fixed by #22548
Assignees
Labels
area:providers good first issue kind:new provider request label to mark request for adding new provider

Comments

@alexInhert
Copy link

Description

https://www.arangodb.com/ is a graph db much like Neo4j.
It has official python connector: https://github.com/ArangoDB-Community/python-arango

Use case / motivation

Using ArangoHook to interact with Arango.
Using ArangoOperator to run simple queries.
Possibly also ArangoAQLHook / ArangoAQLOperator to run AQL queries - not sure if it require separate hook/operator.

@alexInhert alexInhert added the kind:feature Feature Requests label Aug 23, 2021
@potiuk
Copy link
Member

potiuk commented Aug 23, 2021

Would you be willing to add it ? I think if you have the need and use Rango, you might be the best person to add it .

@ashmeet13
Copy link
Contributor

Hi, can I pick this up?

@potiuk
Copy link
Member

potiuk commented Sep 8, 2021

Feel free

@mrbazzan
Copy link
Contributor

@ashmeet13 Are you still working on this issue? I'm interested if you aren't working on it anymore

@ashmeet13
Copy link
Contributor

Hi @mrbazzan, sorry should have updated here - I am not you can go ahead and pick this up. My bad.

I did however start on this and you can find a bit of code that I had written here - https://github.com/ashmeet13/airflow/tree/arango-hook (Only 1 commit was made)

Hope this helps!

@uranusjr uranusjr assigned mrbazzan and unassigned ashmeet13 Oct 19, 2021
@mrbazzan
Copy link
Contributor

@ashmeet13 Oh... Thanks, that'll help.

@vijaya-lakshmi-venkatraman

Hello @mrbazzan If this is still available, I would like to try!

@mrbazzan
Copy link
Contributor

@vijaya-lakshmi-venkatraman
No, not at the moment. If I'm still unable to do something in the coming week, I'll drop it.

@eladkal eladkal added kind:new provider request label to mark request for adding new provider and removed kind:feature Feature Requests labels Mar 18, 2022
@pateash
Copy link
Contributor

pateash commented Mar 18, 2022

@eladkal as discussed over slack, I will pick this up.

@Megha-Sai-Sree-PINNAKA
Copy link

@pateash can I know if you are working on documentation of Arango Hook, else I am interested to take it up.

@pateash
Copy link
Contributor

pateash commented Apr 6, 2022

@pateash can I know if you are working on documentation of Arango Hook, else I am interested to take it up.

I have added some as part of PR,
@eladkal Do we need any more?

@Megha-Sai-Sree-PINNAKA
Copy link

Hello @pateash I see you have documented the development work of Arango Hook. I would like to document the use cases of Arango Hook if it is ok.

@pateash
Copy link
Contributor

pateash commented Apr 7, 2022

Hello @pateash I see you have documented the development work of Arango Hook. I would like to document the use cases of Arango Hook if it is ok.

Sure, go ahead

@Megha-Sai-Sree-PINNAKA
Copy link

Hello. I tried comparing the structure of MongoDB's docs folder with ArangoDB's, it looked almost similar. But then we are able to see MongoDB in the official documentation of apache airflow whereas ArangoDB is not coming up. Is there anything exclusively to be done inorder to see Arango in the documentation https://airflow.apache.org/docs/?

@pateash
Copy link
Contributor

pateash commented May 19, 2022

the docs are available here,
https://airflow.apache.org/docs/apache-airflow-providers-arangodb/1.0.0/

I am looking into, why they are not available on main page.

@pateash
Copy link
Contributor

pateash commented May 19, 2022

I am able to see arangodb, when i build docs locally

image

@potiuk @eladkal , Do we build all the docs or just the provider which is released?

@potiuk
Copy link
Member

potiuk commented May 19, 2022

Because it has to be added manually - this is one of the few steps in process which is still manual and sometimes can be missed:

PR here: apache/airflow-site#597

@potiuk
Copy link
Member

potiuk commented May 19, 2022

Any improvement in the automation there are welcome:

The step is here:

https://github.com/apache/airflow/blob/main/dev/README_RELEASE_PROVIDER_PACKAGES.md#prepare-documentation

If you publish a new package, you must add it to the docs index:

@potiuk
Copy link
Member

potiuk commented May 19, 2022

Link is up on our website: https://airflow.apache.org/docs/

@pateash
Copy link
Contributor

pateash commented May 20, 2022

Link is up on our website: https://airflow.apache.org/docs/

Thanks @potiuk

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:providers good first issue kind:new provider request label to mark request for adding new provider
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants