Neo4j SQL Misnomer

[ThomasThelen]

Apache Airflow Provider(s)

neo4j

Versions of Apache Airflow Providers

No response

Apache Airflow version

2.9.0

Operating System

macOS

Deployment

Docker-Compose

Deployment details

No response

What happened

The Neo4jOperator makes reference to executing sql, when the actual query language is cypher. SQL is not used to query graph databases, and Neo4j doesn't support SQL.

This was confusing to me because I had to make sure that the operator in fact, was using CYPHER instead of sql.

The example here uses CYPHER, but passes it into the sql kwarg which is confusing/misleading/incorrect.

What you think should happen instead

Instead of the operator taking a sql kwarg it should be renamed to cypher to match the actual query language being used.

How to reproduce

Reproduce by looking at the source https://github.com/apache/airflow/blob/main/airflow/providers/neo4j/operators/neo4j.py#L31

Anything else

No response

Are you willing to submit PR?

• Yes I am willing to submit a PR!

Code of Conduct

• I agree to follow this project's Code of Conduct

[boring-cyborg]

Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for approval.

[sandeepvsk10]

Hello, would love to work on this! and some couple other Neo4j feature additions on separate thread. Thanks!