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

Improve IO locality in MultiplePathsFromGCRootsComputerImpl bfs #46

Closed
eclipsewebmaster opened this issue May 8, 2024 · 11 comments
Closed

Comments

@eclipsewebmaster
Copy link

| --- | --- |
| Bugzilla Link | 582821 |
| Status | ASSIGNED |
| Importance | P3 normal |
| Reported | Dec 31, 2023 15:01 EDT |
| Modified | Jan 17, 2024 08:54 EDT |
| Version | 1.15 |

@eclipsewebmaster
Copy link
Author

By Jason Koch on Dec 31, 2023 15:03

https://git.eclipse.org/r/c/mat/org.eclipse.mat/+/205956

@eclipsewebmaster
Copy link
Author

By Andrew Johnson on Jan 03, 2024 14:44

I think this is okay.
Another case of slow I/O is a HPROF file compressed with gzip (not so much with chucked GZIP as generated by the JVM, or by export HPROF with the chunked option). This can involve restarting the gunzip from a PosStream position, and depending on memory this might be more widely spaced than the 1MB chunked multi-memory gzip file.

@eclipsewebmaster
Copy link
Author

By Jason Koch on Jan 16, 2024 17:35

Note that this changes the results presented in the tests because the sort() order changes the traversal order at each BFS layer. Removing the sort during each iteration confirms this.

@eclipsewebmaster
Copy link
Author

By Jason Koch on Jan 16, 2024 17:35

the *regression test suite, that is

@jasonk000
Copy link
Contributor

this was merged in [https://git.eclipse.org/r/c/mat/org.eclipse.mat/+/205956 , this commit.

This is as much as stands out to me for now.

@jasonk000
Copy link
Contributor

This is ready to close @krumts but I don't have perms =)

@krumts krumts closed this as completed May 20, 2024
@krumts
Copy link
Contributor

krumts commented May 20, 2024

but I don't have perms =)

@jasonk000 , this shouldn't be the case. I was able to close it, and I guess/hope every committer on the project should be able too. Not sure if this is something special for the migrated issues. Will try to figure it out

@krumts
Copy link
Contributor

krumts commented May 20, 2024

@jasonk000 I see you are shown as "Contributor" while I am show as "Member" of the eclipse-mat organization. There was an invite after the move. Did you get one? I don't find you under members of the org.

@jasonk000
Copy link
Contributor

I had a look, and I do not see an invite. Checked my spam/deleted as well and did not see it.

@kgibm
Copy link
Member

kgibm commented May 21, 2024

I believe it requires setting your GitHub user ID in the Eclipse developer configuration. Did you do that?

image

This is the invite email I got:

image

@krumts
Copy link
Contributor

krumts commented May 23, 2024

@jasonk000 the web masters confirmed that the link to your github id is missing. Once you link it, you will automatically get the invitation to join (within 2h).

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

No branches or pull requests

4 participants