Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: daf_butler
-
Labels:None
-
Story Points:2
-
Epic Link:
-
Sprint:Arch 2019-05-20, Arch 2019-05-27, Arch 2019-06-03
-
Team:Architecture
Description
In a GitHub pull request Dino Bektesevic makes the case that URIs in the Location classes are not working properly. His fix is not quite right so take the PR and see if it's possible to fix things such that URIs with relative paths are properly handled.
Andy Salnikov Would you be able to review this please? As I say on the PR, I've changed most of the API that was only used within Location and LocationFactory, such that it is now consistently using URIs for the datastore root, and relative paths for the location inside the datastore. This makes things much simpler. The API as used by butler did not change and all tests still pass. The Location classes now should be able to work fine for s3 URIs for example.