[DEPR]: Remove Neo4J Support #34342

[qasimgulzar]

Issue

Proposal Date: 2024-03-07
Target Ticket Acceptance Date: 2024-03-21
Earliest Open edX Named Release Without This Functionality: Sumac 2024-10

Rationale

This pull request addresses the removal of Neo4j support from the edx-platform, along with the associated dependencies and the coursegraph app. The decision to deprecate and remove this functionality stems from the observation that Neo4j usage is underused and not deemed particularly valuable, with many operators transitioning to alternative data stores for course data search.

Changes Made

1. Removal of Neo4j dependencies.
2. Removal of the coursegraph app from cms.djangpapps.coursegraph.

Replacement

There is currently no direct replacement proposed for the functionality being removed. However, users interested in retaining this functionality can explore creating a separate plugin.

Discussion Thread

Discuss thread

Solution Implementation

The solution involves removing the Neo4j dependencies and the coursegraph app from the specified location in the codebase.

Solution Implementation Details:

I have removed the neo4j dependencies as well as the course graph app from cms.djangpapps.coursegraph

[openedx-webhooks]

Thanks for the pull request, @qasimgulzar! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

• supporting documentation
• Open edX discussion forum threads
• timeline information ("this must be merged by XX date", and why that is)
• partner information ("this is a course on edx.org")
• any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

[mphilbrick211]

Hi @openedx/2u-vanguards! Are you able to take a look at this?

[awais786]

@qasimgulzar for safe side you can split this PR into two parts. In first part remove the code and in 2nd PR remove the package.

[qasimgulzar]

@qasimgulzar for safe side you can split this PR into two parts. In first part remove the code and in 2nd PR remove the package.

I would agree with your recommendation, so removed the requirement's changes for this PR.

[feanil]

Looks like 2U is still using this old infrastructure and so we should hold of till May 27, 2024 to merge this PR.

[feanil]

Merging a bit early since we've got the go ahead.

[openedx-webhooks]

@qasimgulzar 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.

[edx-pipeline-bot]

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

[edx-pipeline-bot]

2U Release Notice: This PR has been deployed to the edX production environment.

[edx-pipeline-bot]

2U Release Notice: This PR has been deployed to the edX production environment.