search

saga-project / BigJob

SAGA-based Pilot-Job Implementation for Compute and Data
http://saga-project.github.com/BigJob/
Other
19 stars 8 forks source link

PilotCompute shouldn't set default wall time to 3600 #88

Open oleweidner opened 11 years ago

oleweidner commented 11 years ago

in pilotcompute_manager.py:149:

        walltime = 3600
        if ("walltime" in bj_dict):
            walltime=bj_dict["walltime"]

not a good idea. how is the user supposed to know? 3600 is also way above most queuing system limits. BigJob should simply throw an error if 'walltime' is not set. same for other pilot_compute parameters like 'working_directory' and so on. this would prevent a LOT of confusion and would make debugging BigJob much easier!

oleweidner commented 11 years ago

Has this been fixed?