Thomas Graves

so I would like to see the issue or this PR description have much more details about design, API, and its behavior. For instance: ``` Does this PR introduce any...

thanks for adding more details. ``` /** * Return resource profile Ids of executors where tasks can be assigned to. */ def compatibleExecutorRpIds(rpMgr: ResourceProfileManager): Set[Int] ``` It seems a little...

> Yes, exactly. I put the ResourceProfileManager here because it knows about all the ResourceProfiles. > Adding this API just want to make sure that we have one interface to...

sorry , might be next Monday before I get a chance.

yes, it would be nice to have the actual stagIds in the ui, I'll need to look closer at the logic though, which likely won't be til next week.

So a couple questions and concerns. 1. I'm not sure having the list of Completed Stages at the top of the page helps and I'm concerned that could be a...

sorry, I missed your response. After looking some more the list of jobs can get very long as well. I'm fine with leaving the list of stages as well. Do...

> Does this PR introduce any user-facing change? > No Is this true? If I specify a file to pass along and my code references it, what did it do...

items 2 and 3 in the description, is one of those supposed to be client mode? Otherwise they are the same.

can you try kicking the tests again?