github / stack-graphs

Rust implementation of stack graphs
https://docs.rs/stack-graphs/*/stack_graphs/
Apache License 2.0
717 stars 122 forks source link

Errors can report non-existent tree/path combinations #398

Open robrix opened 5 months ago

robrix commented 5 months ago

I've seen a few errors come through Splunk (cf discussion in Slack) where the given repo_id/commit_oid combination exists, but doesn't have the repo_path—or in many cases anything even resembling the package_path at that or any other SHA in the repo. @hendrikvanantwerpen discovered that in at least one of these cases, the path in question does exist elsewhere in the repo's network. I guess everything really is somewhere. It's unknown whether that applies to all cases.

Ultimately, we're left with a few questions:

I happen to only have seen this with JS, but that doesn't mean it's not happening with other languages or indexing strategies; I simply do not know.

hendrikvanantwerpen commented 4 months ago

I suspect this is an issue in Aleph instead of stack graphs. Perhaps we should transfer it?