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

Replace custom org.dcache.util.list.DirectoryStream with java.nio.file.DirectoryStream #3821

Open
kofemann opened this issue Feb 12, 2018 · 1 comment · May be fixed by #7262
Open

Replace custom org.dcache.util.list.DirectoryStream with java.nio.file.DirectoryStream #3821

kofemann opened this issue Feb 12, 2018 · 1 comment · May be fixed by #7262
Labels
easypick education

Comments

@kofemann
Copy link
Member

Before java7 was available dcache have added org.dcache.util.list.DirectoryStream which is made as close as possible to java.nio.file.DirectoryStream. Since java7 is already released for years, we can finally drop custom interface and adopt it's the JDK provided analogue.
@JPOstholt
Copy link

will do

JPOstholt added a commit to JPOstholt/dcache that referenced this issue Feb 13, 2019
@JPOstholt
dcache: Replaced org.dcache.util.list.DirectoryStream with java.nio.f…
865e505 
…ile.DirectoryStream

Motivation: Since java7 is already released for years, we can finally drop the custom interface and adopt it's JDK provided analogue

Modification: Removed org.dcache.util.list.DirectoryStream and replaced it with java.nio.file.DirectoryStream, adding a few catch clauses and generic type casts

Result: dcache now uses java.nio.file.DirectoryStream and org.dcache.util.list.DirectoryStream could be deleted

Fixes: #gh-issue
Ticket: dCache#3821
Target: master

Signed-off-by: JPOstholt <jascha.ostholt@gmail.com>
JPOstholt added a commit to JPOstholt/dcache that referenced this issue Feb 13, 2019
@JPOstholt
dcache: Replaced org.dcache.util.list.DirectoryStream with java.nio.f…
db89517 
…ile.DirectoryStream

Motivation: Since java7 is already released for years, we can finally drop the custom interface and adopt it's JDK provided analogue

Modification: Removed org.dcache.util.list.DirectoryStream and replaced it with java.nio.file.DirectoryStream, adding a few catch clauses and generic type casts

Result: dcache now uses java.nio.file.DirectoryStream and org.dcache.util.list.DirectoryStream could be deleted

Fixes: #gh-issue
Ticket: dCache#3821
Target: master

Signed-off-by: JPOstholt <jascha.ostholt@gmail.com>
@LukasMansour LukasMansour linked a pull request Aug 1, 2023 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
easypick education
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Replace dCache's 'DirectoryStream' with Java's 'DirectoryStream' LukasMansour/dcache
2 participants
@kofemann @JPOstholt