Skip to content

Rename Neo4jOperator sql parameter to cypher#68970

Merged
eladkal merged 3 commits into
apache:mainfrom
eladkal:4j
Jun 25, 2026
Merged

Rename Neo4jOperator sql parameter to cypher#68970
eladkal merged 3 commits into
apache:mainfrom
eladkal:4j

Conversation

@eladkal

@eladkal eladkal commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Cypher is Neo4j's query language, not SQL. The sql parameter still works but now raises a deprecation warning. closes: #39860

Note cypher parameter is already being used in Neo4jSensor so I kept this convention


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

Cypher is Neo4j's query language, not SQL. The `sql` parameter still works
but now raises a deprecation warning. closes: apache#39860
@eladkal
eladkal requested review from o-nikolas and vincbeck June 25, 2026 16:47
@eladkal
eladkal merged commit 30a7071 into apache:main Jun 25, 2026
77 checks passed
@eladkal
eladkal deleted the 4j branch June 25, 2026 17:45
karenbraganz pushed a commit to karenbraganz/airflow that referenced this pull request Jun 30, 2026
* Rename Neo4jOperator `sql` parameter to `cypher`

Cypher is Neo4j's query language, not SQL. The `sql` parameter still works
but now raises a deprecation warning. closes: apache#39860

* fix

* Unify examples into one file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Neo4j SQL Misnomer

2 participants