-
Notifications
You must be signed in to change notification settings - Fork 0
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
Fix finding topic paths for s3 #74
Conversation
… implementation based on ObjectName
I'm wondering if you could call |
@blootsvoets Yes. I gave it a try yesterday. It didn't work. I had a better look on it today.
The result is
|
Using a slash after the path name fixes the infinite recursion. Making lastModified optional fixes the nullpointerexception. |
Thanks. Adding the |
I will proceed with the merge since you have made your changes @blootsvoets . |
Current implementation processes only the first topic listed in the objects lists for s3 implementation.
s3 client's listObject() only lists available objects. It doesn't list directories as separate item. Hence
isDir
is also always false.This fix allows to have the findTopicPaths() function overridden for s3 and old implementation for other
SourceStorage
.