Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Oct 30, 2023. It is now read-only.

Conversation

@reudismam
Copy link

No description provided.

@dlogothetis
Copy link
Contributor

@reudismam, can you create a JIRA issue on https://issues.apache.org/jira/projects/GIRAPH/ for this and your other pull requests? Also, could you add in the description of the PRs how you tested them? Thanks!

@dlogothetis
Copy link
Contributor

Can also describe why this change is necessary?

@reudismam
Copy link
Author

reudismam commented Nov 4, 2018

Strings can be used to represent a file system path even though some classes are specifically designed for this task. For instance, java.nio.Path. It is useful to change the type of the variable to Path, since strings can be combined in an undisciplined way, which can lead to invalid paths. Second, different operating systems use different file separators, which can cause bugs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants