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

Update moved procedures to have new class names #4257

Merged
merged 2 commits into from
Nov 29, 2024
Merged

Conversation

gem-neo4j
Copy link
Contributor

This is needed in the 5.26 branch, or if someone has both Core and Extended installed, there will be missing procedures!

Copy link
Collaborator

@vga91 vga91 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice :)
Just some Extended here as well

@@ -41,7 +41,7 @@
* @author moxious
* @since 27.02.19
*/
public class Neo4jLogStream {
public class Neo4jLogStreamExtended {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Extended annotation

@@ -45,7 +45,7 @@
import java.util.Map;
import java.util.stream.Stream;

public class ExportArrow {
public class ExportArrowExtended {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Extended annotation

@@ -46,8 +46,7 @@ public class LoadJsonExtended {
public URLAccessChecker urlAccessChecker;

@SuppressWarnings("unchecked")
@Procedure(name = "apoc.load.jsonParams", deprecatedBy = "This procedure is being moved to APOC Extended.")
@Deprecated
@Procedure(name = "apoc.load.jsonParams")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LoadJsonExtended with @Extended as well

@gem-neo4j gem-neo4j merged commit 0444b75 into dev Nov 29, 2024
1 of 5 checks passed
@gem-neo4j gem-neo4j deleted the dev_clashing_names_fix branch November 29, 2024 11:39
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.

2 participants