-
Notifications
You must be signed in to change notification settings - Fork 4
Conversation
mariner/tool.go
Outdated
path, err := filePath(v) | ||
if err != nil { | ||
tool.Task.infof("failed to extract path from file: %v", v) | ||
continue |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should not continue here, instead we should return the error
mariner/tool.go
Outdated
path, err := filePath(f) | ||
if err != nil { | ||
tool.Task.infof("failed to extract path from file: %v", f) | ||
continue |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not continue, but return error
mariner/tool.go
Outdated
switch x := output.(type) { | ||
case *File: | ||
path := output.(*File).Path | ||
tool.Task.infof("*File - Path: %v", path) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
currently i dont handle this case, so either we handle it here or we just remove this and the []*File
case below to be caught in default
and throw the exception
mariner/tool.go
Outdated
continue | ||
} | ||
switch { | ||
case strings.HasPrefix(path, userPrefix): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this series of case logic is repeated in many different areas here so perhaps should be turned into a function?
engine.IsInitWorkDir = "true" | ||
tool.Task.infof("s3input paths: %v", tool.S3Input.Paths) | ||
tool.Task.infof("initWorkDirFiles: %v", tool.initWorkDirFiles) | ||
continue |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I dislike how this function has to use the continue
for this situation
b0ce0a3
to
b8a463c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
b8a463c
to
3ba7f37
Compare
3ba7f37
to
f945835
Compare
Jira Ticket: PXP-8788 PXP-8788 PXP-8843
Feature
We have via this PR fixed a variety of bugs described in the tickets above. Also by fixing these bugs we are finally able to run the pre-genesis workflow.
Bug fix