-
-
Notifications
You must be signed in to change notification settings - Fork 706
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
Incorrect topological sorting leads to incorrect SCC computation in Spark #2080
Comments
This appears to have been broken by #1843 |
michael-emmi
pushed a commit
to michael-emmi/soot
that referenced
this issue
May 15, 2024
michael-emmi
pushed a commit
to michael-emmi/soot
that referenced
this issue
May 15, 2024
michael-emmi
pushed a commit
to michael-emmi/soot
that referenced
this issue
May 15, 2024
michael-emmi
pushed a commit
to michael-emmi/soot
that referenced
this issue
May 15, 2024
michael-emmi
pushed a commit
to michael-emmi/soot
that referenced
this issue
May 16, 2024
michael-emmi
pushed a commit
to michael-emmi/soot
that referenced
this issue
May 16, 2024
michael-emmi
pushed a commit
to michael-emmi/soot
that referenced
this issue
May 16, 2024
michael-emmi
pushed a commit
to michael-emmi/soot
that referenced
this issue
May 16, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Please examine each of the following points so that we can help you as soon and best as possible.
Describe the bug
Spark’s SCC-collapsing optimization merges PAG nodes which should be in separate components.
Input file
To reproduce
The second assertion fails in the test above.
Expected behavior
The test above should pass, since nodes a, b, and c should each be in their own component.
Stacktrace
N/A
Additional context
It appears that the SCC computation, following Kosaraju’s algorithm, is based on an invalid topological order. The topological sort should order node b before c. Instead, c gets ordered before b. This results in c being the SCC-root for b.
The text was updated successfully, but these errors were encountered: