Task #8620

Include timestamps in AtoM worker log

Added by José Raddaoui Marín almost 7 years ago. Updated almost 7 years ago.

Status:VerifiedStart date:06/18/2015
Priority:LowDue date:
Assignee:José Raddaoui Marín% Done:

0%

Category:Job scheduling
Target version:Release 2.2.0
Google Code Legacy ID: Tested version:
Sponsored:No Requires documentation:

History

#2 Updated by José Raddaoui Marín almost 7 years ago

  • Status changed from New to Code Review
  • Assignee changed from José Raddaoui Marín to Mike Gale

PR 199

Hi Mike, I've removed the ->info and ->err calls, I think the messages should be enough. Let me know if you think otherwise, maybe we could add something in the formatLogMsg() function.

#3 Updated by José Raddaoui Marín almost 7 years ago

  • Status changed from Code Review to QA/Review
  • Target version changed from Release 2.3.0 to Release 2.2.0

#4 Updated by Mike Gale almost 7 years ago

  • Assignee changed from Mike Gale to José Raddaoui Marín

looks good

#5 Updated by José Raddaoui Marín almost 7 years ago

  • Assignee changed from José Raddaoui Marín to Mike Gale

I did a quick test locally after merging it in qa/2.2.x and it's working good.

We need to check that the following line is not removed when we merge qa/2.2.x in qa/2.3.x:

https://github.com/artefactual/atom/blob/qa/2.3.x/lib/job/arBaseJob.class.php#L38

#6 Updated by Mike Gale almost 7 years ago

  • Status changed from QA/Review to Verified
  • Assignee changed from Mike Gale to José Raddaoui Marín

#7 Updated by Mike Gale almost 7 years ago

works fine here

Also available in: Atom PDF