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

python_dep_local_graph case sensitivity #2

Open
Vaults opened this issue Oct 21, 2022 · 1 comment
Open

python_dep_local_graph case sensitivity #2

Vaults opened this issue Oct 21, 2022 · 1 comment

Comments

@Vaults
Copy link
Contributor

Vaults commented Oct 21, 2022

Hi,

Unfortunately the neo4j database in https://github.com/wagenrace/python_dep_local_graph has some quirks with case sensitivity. It seems that the Package column has a capitalized first letter on Windows, whereas it has a non-capitalized first letter on Linux. This leads to neo4j having a lower case package identifier on Linux (production) and the query breaking.

Proposed fix:

  • Force lowercase on the identifiers on import somehow
  • Change MATCH (n:Package)-[:DEPENDS_ON*0..]->(m:Package) to MATCH (n:package)-[:DEPENDS_ON*0..]->(m:package)
    in line
    MATCH (n:Package)-[:DEPENDS_ON*0..]->(m:Package)
    as this is currently the case in production.
@wagenrace
Copy link
Owner

Thanks, I indeed made mistake in the local version. Upper case is the convention for labels

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants