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.

Sign up for GitHub

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

Jump to bottom

mvnd --status throws NumberFormatException in 0.0.9 #147

Closed
odrotbohm opened this issue Oct 25, 2020 · 5 comments · Fixed by #149
Closed

mvnd --status throws NumberFormatException in 0.0.9 #147

odrotbohm opened this issue Oct 25, 2020 · 5 comments · Fixed by #149
Milestone
0.0.10

Comments

@odrotbohm
Copy link

odrotbohm commented Oct 25, 2020 •
edited
Loading 

$ mvnd …
…
$ mvnd --status
                                    UUID      PID   Port   Status    RSS            Last activity  Java home
Exception in thread "main" java.lang.NumberFormatException: For input string: " 84276"
	at java.lang.Long.parseLong(Long.java:678)
	at java.lang.Long.parseLong(Long.java:817)
	at org.jboss.fuse.mvnd.common.OsUtils.findProcessRssInKb(OsUtils.java:59)
	at org.jboss.fuse.mvnd.client.DefaultClient.lambda$execute$0(DefaultClient.java:149)
	at java.util.ArrayList.forEach(ArrayList.java:1541)
	at org.jboss.fuse.mvnd.client.DefaultClient.execute(DefaultClient.java:147)
	at org.jboss.fuse.mvnd.client.DefaultClient.main(DefaultClient.java:72)

Looks like there's a String.trim() missing?
@ppalaga
Copy link
Contributor

ppalaga commented Oct 26, 2020

Thanks for the report, @odrotbohm

Which OS is that?

@odrotbohm
Copy link
Author

odrotbohm commented Oct 26, 2020 •
edited
Loading

MacOS Catalina.

@ppalaga
Copy link
Contributor

ppalaga commented Oct 26, 2020

Yeah, .trim() is not there. Let me add it. Strange that the test that we have for this is passing on GH Action's Catalina.

ppalaga added a commit to ppalaga/mvnd that referenced this issue Oct 26, 2020
@ppalaga
mvnd --status throws NumberFormatException in 0.0.9 apache#147
740b5f1
@ppalaga ppalaga mentioned this issue Oct 26, 2020
Merged
ppalaga added a commit that referenced this issue Oct 26, 2020
@ppalaga
mvnd --status throws NumberFormatException in 0.0.9 #147
73e8a9a
@ppalaga
Copy link
Contributor

ppalaga commented Oct 26, 2020

@odrotbohm you may want to test/use the package built by the CI https://github.com/mvndaemon/mvnd/suites/1398611867/artifacts/23216845 before we release.

@ppalaga ppalaga added this to the 0.0.10 milestone Oct 26, 2020
@odrotbohm
Copy link
Author

Playing with the snapshot build, looks like this is fixed. Thanks!

@jglick jglick mentioned this issue Mar 17, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.0.10
Development

Successfully merging a pull request may close this issue.

mvnd --status throws NumberFormatException in 0.0.9 ppalaga/mvnd
2 participants
@odrotbohm @ppalaga