When scheduling a job, should not override the old one if exists

• Type: Bug
• Status: Closed
• Resolution: Fixed
• Affects Version/s: 7.0.X EE, Master
• Fix Version/s: 7.0.0 DXP FP28, 7.0.X EE, 7.0.4 CE GA5, 7.1.X, Master
• Component/s: Portal Services > Scheduler
• Labels:
  • liferay-digital-enterprise-70-sp5
  • liferay-fixpack-de-28-7010
• Branch Version/s:
  7.0.x
• Backported to Branch:
  Committed
• Git Pull Request: https://github.com/brianchandotcom/liferay-portal/pull/50893

This is found by analyzing code, can not be tested manually.

In old logic, when we schedule a job on Quartz, we do :

synchronized (this) {
scheduler.deleteJob(trigger.getJobKey());
scheduler.scheduleJob(jobDetail, trigger);
}

We should not do this, since we already provide a way to update a job, what we should do is , do not override old ones, and do some logging if job with same name exists