[jira] [Closed] (SLING-7529) Log message layouts are not property inherited

2018-03-26 Thread Chetan Mehrotra (JIRA)

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

Chetan Mehrotra closed SLING-7529.
--

> Log message layouts are not property inherited
> --
>
> Key: SLING-7529
> URL: https://issues.apache.org/jira/browse/SLING-7529
> Project: Sling
>  Issue Type: Bug
>  Components: Commons
>Affects Versions: Commons Log 5.1.2
>Reporter: Fryderyk Wysocki
>Assignee: Chetan Mehrotra
>Priority: Major
> Fix For: Commons Log 5.1.4
>
>
> Steps to reproduce on AEM 6.3 with Apache Sling Commons Log 5.1.2:
> 1. Create Logger Configuration for Logger "my.project", log file "error.log", 
> log level 'Information" and message pattern "My project: \{5}"
> 2. Create an Slf4J logger for class my.project.Sample and use it to infolog 
> message "test foo"
> 3. Change the configuration created in step 1 - update logger to 
> "my.project.Sample"
> 4. Use the logger created in step 2 to infolog message "test bar"
> Expected result:
> 5. error.log file would contain messages "My project: test foo" and "My 
> project: test bar"
> Actual result:
> 5. error.log file contains the message "test foo" in the format configured as 
> the default format, but also contains the message "My project: test bar" - 
> the second message, in the appropriate format.
> Investigation:
> Checking the source code, I've discovered the following in class 
> 'LoggerSpecificEncoder', starting from line 47:
> {code:java}
> private Layout getLayout(String loggerName) {
> // TODO Handle layout inheritance wrt logger names
> Layout layout = layoutByCategory.get(loggerName);
> if (layout == null) {
> layout = defaultLayout;
> }
> return layout;
> }
> {code}
> Which means that, unless the configuration is created for the exact logger 
> name, the default one will be used - the functionality is not broken, but 
> simply not implemented.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[RESULT] [VOTE] Release Apache Sling Commons Log 5.1.4

2018-03-26 Thread Chetan Mehrotra
Hi,

The vote has passed with the following result :

+1 (binding): Stefan Seifert, Carsten Ziegeler, Daniel Klco, Chetan Mehrotra

I will copy this release to the Sling dist directory and promote the
artifacts to the central Maven repository.

Chetan Mehrotra


[jira] [Created] (SLING-7555) Remove ghost packages from distribution agent queue

2018-03-26 Thread Marius Petria (JIRA)
Marius Petria created SLING-7555:


 Summary: Remove ghost packages from distribution agent queue
 Key: SLING-7555
 URL: https://issues.apache.org/jira/browse/SLING-7555
 Project: Sling
  Issue Type: Improvement
  Components: Content Distribution
Affects Versions: Content Distribution Core 0.2.10
Reporter: Marius Petria
 Fix For: Content Distribution Core 0.3.0


Currently, (see [1]) an agent exporter only logs an error when a "ghost 
package" is encountered (a queue entry without a binary). We should log this as 
an error and remove the item from the queue in order to avoid stuck queues.

 

 

[1]https://github.com/apache/sling-org-apache-sling-distribution-core/blob/master/src/main/java/org/apache/sling/distribution/packaging/impl/exporter/AgentDistributionPackageExporter.java#L102



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)