Skip to content
This repository has been archived by the owner on Jan 9, 2020. It is now read-only.

Failing submission if submitter-local files are provided, but no reso… #362

Closed
wants to merge 2 commits into from

Conversation

sahilprasad
Copy link

…urce staging URI is given (#339)

Validating the behavior as per the issue.

@foxish
Copy link
Member

foxish commented Jun 29, 2017

/cc @mccheah @ash211

@foxish
Copy link
Member

foxish commented Jul 7, 2017

This will likely not be applicable following the major redesign in #365

@mccheah
Copy link

mccheah commented Jul 7, 2017

@foxish It's still applicable but it needs to be written into the new design. We can punt on this PR for now and re-write it with the new design - shouldn't be too difficult.

@mccheah
Copy link

mccheah commented Jul 7, 2017

Or we can merge as-is and rewrite it into the new design PR.

@erikerlandson
Copy link
Member

@foxish @mccheah did this get folded into the submission refactoring?

@mccheah
Copy link

mccheah commented Jul 26, 2017

Ah, I don't think so - can this be rebased and re-worked accordingly?

@sahilprasad
Copy link
Author

@mccheah sure thing. Since it's such a small fix, I'll close this PR for now, and open another once I've refactored.

ifilonenko pushed a commit to ifilonenko/spark that referenced this pull request Feb 26, 2019
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.

None yet

4 participants