[ 
https://issues.apache.org/jira/browse/SLING-10066?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Timothee Maret reopened SLING-10066:
------------------------------------

Reopening since 
https://github.com/apache/sling-org-apache-sling-distribution-journal/commit/dd655d8de4189b5e43ddbc54342dfade9ccdde3b
 introduces a bug that prevent distributing content.

{code}
12.03.2021 12:36:57.070 *ERROR* [127.0.0.1 [1615552616884] POST 
/libs/sling/distribution/services/agents/publish HTTP/1.1] 
org.apache.sling.distribution.journal.impl.publisher.DistributionPublisher 
[null] Failed to queue distribution request null
java.util.NoSuchElementException: null
        at java.base/java.util.ArrayList$Itr.next(ArrayList.java:999)
        at 
org.apache.sling.distribution.journal.impl.publisher.DistributionPublisher.execute(DistributionPublisher.java:277)
 [org.apache.sling.distribution.journal:0.1.17.T202103112338-51c5f71]
        at 
org.apache.sling.distribution.journal.impl.publisher.DistributionPublisher.execute(DistributionPublisher.java:258)
 [org.apache.sling.distribution.journal:0.1.17.T202103112338-51c5f71]
        at 
org.apache.sling.distribution.servlet.DistributionAgentServlet.doPost(DistributionAgentServlet.java:62)
 [org.apache.sling.distribution.core:0.4.5.T202103112324-7352c87]
        at 
org.apache.sling.api.servlets.SlingAllMethodsServlet.mayService(SlingAllMethodsServlet.java:146)
 [org.apache.sling.api:2.22.0]
        at 
org.apache.sling.api.servlets.SlingSafeMethodsServlet.service(SlingSafeMethodsServlet.java:342)
 [org.apache.sling.api:2.22.0]
        at 
org.apache.sling.api.servlets.SlingSafeMethodsServlet.service(SlingSafeMethodsServlet.java:374)
 [org.apache.sling.api:2.22.0]
{code}

> cleanup logging for content distribution
> ----------------------------------------
>
>                 Key: SLING-10066
>                 URL: https://issues.apache.org/jira/browse/SLING-10066
>             Project: Sling
>          Issue Type: Improvement
>          Components: Content Distribution
>    Affects Versions: Content Distribution Journal Core 0.1.16
>            Reporter: Joerg Hoh
>            Assignee: Timothée Maret
>            Priority: Major
>             Fix For: Content Distribution Journal Core 0.1.18
>
>
> When submitting content for distribution I get these 4 log entries for a 
> single submission (using the package-id as identifier):
> {code:java}
> 15.01.2021 14:30:05.168 *INFO* [192.147.128.10 [1610721005131] POST 
> /bin/replicate.json HTTP/1.1] 
> org.apache.sling.distribution.journal.impl.publisher.PackageMessageFactory 
> Creating package binary with id [f4fce37b-f160-40b0-9284-7b696a954630] for 
> package [dstrpck-1610721005168-00bd582b-1f91-4abc-82b6-a80afa2a0483], length 
> [8343]
> 15.01.2021 14:30:05.243 *INFO* [Message Poller PackageMessage handled by 
> MessagingCacheCallback$$Lambda$532/0x000000080108fc40] 
> org.apache.sling.distribution.journal.queue.impl.PubQueueCache Queueing 
> message 
> package-id=dstrpck-1610721005168-00bd582b-1f91-4abc-82b6-a80afa2a0483, 
> offset=1023083
> 15.01.2021 14:30:17.716 *INFO* [sling-default-1-Registered Service.5223] 
> org.apache.sling.distribution.journal.impl.publisher.PackageDistributedNotifier
>  Sending distributed notifications for pub agent publish queue item 
> dstrpck-1610721005168-00bd582b-1f91-4abc-82b6-a80afa2a0483
> 15.01.2021 14:30:17.779 *INFO* [sling-default-1-Registered Service.5223] 
> org.apache.sling.distribution.journal.impl.publisher.DistributionPublisher 
> [null] Succesfully applied package with id 
> dstrpck-1610721005168-00bd582b-1f91-4abc-82b6-a80afa2a0483, type ADD, paths 
> [/content/something]
> {code}
> While they are technically perfectly ok, I think that 4 messages are a bit 
> too much. I would suggest to change the loglevel for message 3 to DEBUG, 
> because it does not offer any useful information, which should be logged on 
> INFO.
>  I would also suggest to change message 4 to DEBUG (the fact that the package 
> is successfully distributed is implicit, because otherwise there would be an 
> error message), but I am indeed interested into the path which is 
> distributed. So it would be great if we could log that path(s) in either in 
> message 1 or 2.
> Overall this could reduce the noise in the logs if distribution is heavily 
> used.
> WDYT?



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to