Mohit Arora created SLING-10178:
-----------------------------------
Summary: Queueitem should be moved to error queue if the
distribution package does not exist anymore
Key: SLING-10178
URL: https://issues.apache.org/jira/browse/SLING-10178
Project: Sling
Issue Type: Bug
Components: Content Distribution
Affects Versions: Content Distribution Core 0.4.2
Reporter: Mohit Arora
There can be a case when the distribution package has been processed but the
queueitem has not been removed properly and is being re-processed. In such a
case (when the associated distribution package is null), [the queue item is
simply
removed|https://github.com/apache/sling-org-apache-sling-distribution-core/blob/master/src/main/java/org/apache/sling/distribution/agent/impl/SimpleDistributionAgentQueueProcessor.java#L159-L162].
Thus, the user does not have the ability to check what happened. If a
queueitem was sent to processing without the corresponding distribution
package, then something must have errored, either the package was distributed
successfully but the queueitem was not [removed successfully by the
QueueProcessor|https://github.com/apache/sling-org-apache-sling-distribution-core/blob/master/src/main/java/org/apache/sling/distribution/queue/impl/simple/SimpleDistributionQueueProcessor.java#L55]
or some other code bug. In such a case we can consider moving the queueitem to
error queue instead of just removing it with an error log.
cc - [~ashishc]
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
