search

ajvb / kala

Modern Job Scheduler
MIT License
2.13k stars 187 forks source link

Fix #41 #129

Closed alpert closed 6 years ago

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.2%) to 81.35% when pulling 5c0ead2073ba724ed611e46357e409636de94a9a on alpert:master into 971d57fbac9ecc80e78e0accdfc743b7f964dd59 on ajvb:master.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.2%) to 81.35% when pulling 6215c394bd6b2efaebb21da82e23fab357172653 on alpert:master into 971d57fbac9ecc80e78e0accdfc743b7f964dd59 on ajvb:master.

alpert commented 7 years ago

Hi. I just updated request acording to your comments. Can you check please?

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 81.172% when pulling 0cd82dd22b0cdc1846dcedb28d368bcb8b4f8bfa on alpert:master into 971d57fbac9ecc80e78e0accdfc743b7f964dd59 on ajvb:master.

alpert commented 7 years ago

Done.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.2%) to 81.35% when pulling 909cb9e56262f9e3265935c1dec1ae656e25e49d on alpert:master into 971d57fbac9ecc80e78e0accdfc743b7f964dd59 on ajvb:master.

ajvb commented 7 years ago

@alpert Sorry for the delay.

I am a big fan of this, and it looks really solid, but there are no tests and there is no storage of the JobStat structs associated with a Job. Any possibility those two things could be added?

Let me know if you need any help with either of those.

gwoo commented 7 years ago

Would definitely be nice to have postgres support, but still waiting for an update on JobStat and more tests.