set SLURM_PRIO_PROCESS environment variable

natefoo / slurm-drmaa

DRMAA for Slurm: Implementation of the DRMAA C bindings for Slurm

GNU General Public License v3.0

48 stars 22 forks source link

Closed anthropotron closed 5 years ago

lparsons commented 5 years ago

Related to issue #18.

natefoo commented 5 years ago

Thanks for the submission! Two concerns:

It's being set in the process's environment, which means it persists outside the context of the job being submitted.
Because of the previous issue, I don't think it'd be thread safe.

I am working on these issues in the always-add-envvars branch.

lparsons commented 5 years ago

Any updates on this? I'm still running into the occasional issue due to it.

natefoo commented 5 years ago

Interesting, I thought I'd finished this. I'll pick this back up ASAP, should be within the next week or so.

natefoo commented 5 years ago

Ok, I implemented this in #23. Thanks for the submission though, @anthropotron!