[jira] [Updated] (SLING-5424) MBeanServer reference in MetricServiceImpl should be made optional

2016-01-12 Thread Chetan Mehrotra (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5424?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Chetan Mehrotra updated SLING-5424: --- Affects Version/s: Commons Metrics 1.0.0 > MBeanServer reference in MetricServiceImpl should

[jira] [Updated] (SLING-5424) MBeanServer reference in MetricServiceImpl should be made optional

2016-01-12 Thread Chetan Mehrotra (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5424?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Chetan Mehrotra updated SLING-5424: --- Description: Currently {{MetricServiceImpl}} has a required dependency on MBeanServer.

[jira] [Resolved] (SLING-5424) MBeanServer reference in MetricServiceImpl should be made optional

2016-01-12 Thread Chetan Mehrotra (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5424?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Chetan Mehrotra resolved SLING-5424. Resolution: Fixed Done with 1724354 > MBeanServer reference in MetricServiceImpl should be

[jira] [Updated] (SLING-5424) MBeanServer reference in MetricServiceImpl should be made optional

2016-01-12 Thread Chetan Mehrotra (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5424?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Chetan Mehrotra updated SLING-5424: --- Issue Type: Bug (was: Improvement) > MBeanServer reference in MetricServiceImpl should be

[jira] [Created] (SLING-5424) MBeanServer reference in MetricServiceImpl should be made optional

2016-01-12 Thread Chetan Mehrotra (JIRA)
Chetan Mehrotra created SLING-5424: -- Summary: MBeanServer reference in MetricServiceImpl should be made optional Key: SLING-5424 URL: https://issues.apache.org/jira/browse/SLING-5424 Project: Sling

Re: MetricServiceTest., why this @After method?

2016-01-12 Thread Chetan Mehrotra
Opened SLING-5424 for that Chetan Mehrotra On Tue, Jan 12, 2016 at 3:24 PM, Chetan Mehrotra wrote: > Indeed its strange that I used @After! MetricsServiceImpl has a > required dependency on MBeanServer and earlier OsgiMock does not > activate the service if

Re: [VOTE] Release Apache Sling Commons Metrics 1.0.0

2016-01-12 Thread Ian Boston
Hi, Is your gpg public key committed somewhere ? pgp.mit.edu does not have a signing record, and since you have done many Oak releases I amd sure your key has been signed. https://pgp.mit.edu/pks/lookup?search=0x88E59E02=vindex Otherwise +1 Best Regards Ian On 12 January 2016 at 07:11, Chetan

[jira] [Closed] (SLING-5171) Optimise last modified information retrieval in UnitChangeMonitor

2016-01-12 Thread Radu Cotescu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5171?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Radu Cotescu closed SLING-5171. --- > Optimise last modified information retrieval in UnitChangeMonitor >

Re: [VOTE] Release Apache Sling Commons Metrics 1.0.0

2016-01-12 Thread Bertrand Delacretaz
On Tue, Jan 12, 2016 at 6:17 AM, Chetan Mehrotra wrote: > ..Staging repository: > https://repository.apache.org/content/repositories/orgapachesling-1406 .. +1 for the release of sha1 (org.apache.sling.commons.metrics-1.0.0-source-release.zip)=

MetricServiceTest., why this @After method?

2016-01-12 Thread Bertrand Delacretaz
Hi (Chetan mostly), Is this code in in MetricServiceTest. [1] useful? @After public void registerMBeanServer() { context.registerService(MBeanServer.class, ManagementFactory.getPlatformMBeanServer()); } The tests pass for me without it, and registering a service after the

Re: [VOTE] Release Apache Sling Commons Metrics 1.0.0

2016-01-12 Thread Chetan Mehrotra
On Tue, Jan 12, 2016 at 2:06 PM, Ian Boston wrote: > Is your gpg public key committed somewhere ? > pgp.mit.edu does not have a signing record, and since you have done many > Oak releases I amd sure your key has been signed. I have so far registered it at pgp.com [1] and per

[RESULT][VOTE] Release Apache Sling Scripting Sightly Engine 1.0.10

2016-01-12 Thread Radu Cotescu
Hello, The voting passed with 3 +1 from Daniel Klco, Oliver Lietz and Stefan Seifert. I'll start promoting the release in a few minutes. Can the PMC please move the artifacts to dist? Thanks, Radu

[jira] [Closed] (SLING-5334) Improve Sightly script resolution for components using inheritance

2016-01-12 Thread Radu Cotescu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5334?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Radu Cotescu closed SLING-5334. --- > Improve Sightly script resolution for components using inheritance >

[jira] [Closed] (SLING-5411) Sightly formatting filter only replaces first 10 placeholders

2016-01-12 Thread Radu Cotescu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5411?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Radu Cotescu closed SLING-5411. --- > Sightly formatting filter only replaces first 10 placeholders >

[jira] [Closed] (SLING-5408) Some Sightly Java Use objects stored in the repository are not recompiled on change

2016-01-12 Thread Radu Cotescu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5408?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Radu Cotescu closed SLING-5408. --- > Some Sightly Java Use objects stored in the repository are not recompiled on > change >

[jira] [Closed] (SLING-5409) Enhance the SightlyJavaCompilerService to not enforce a package name for Use objects stored in the repository

2016-01-12 Thread Radu Cotescu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5409?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Radu Cotescu closed SLING-5409. --- > Enhance the SightlyJavaCompilerService to not enforce a package name for Use > objects stored in the

Re: MetricServiceTest., why this @After method?

2016-01-12 Thread Chetan Mehrotra
Indeed its strange that I used @After! MetricsServiceImpl has a required dependency on MBeanServer and earlier OsgiMock does not activate the service if MBeanServer service is not registered. Hence the "intention" was to get it registered before activation Looks like it does not fail which I

Re: [RESULT][VOTE] Release Apache Sling Scripting Sightly Engine 1.0.10

2016-01-12 Thread Bertrand Delacretaz
On Tue, Jan 12, 2016 at 10:41 AM, Radu Cotescu wrote: > ...Can the PMC please move the artifacts to dist?... Ok, I'll do that now. -Bertrand

[jira] [Commented] (SLING-5416) Thread Pool should stop "gracefully"

2016-01-12 Thread Thomas Mueller (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15093915#comment-15093915 ] Thomas Mueller commented on SLING-5416: --- > backwards compatibility issues What we could do is test

Re: [VOTE] Release Apache Sling Commons Metrics 1.0.0

2016-01-12 Thread Daniel Klco
+1 On Tue, Jan 12, 2016 at 4:40 AM, Chetan Mehrotra wrote: > On Tue, Jan 12, 2016 at 2:06 PM, Ian Boston wrote: > > Is your gpg public key committed somewhere ? > > pgp.mit.edu does not have a signing record, and since you have done many > > Oak

Re: [VOTE] Release Apache Sling Scripting Groovy 1.0.2

2016-01-12 Thread Daniel Klco
+1 On Mon, Jan 11, 2016 at 4:16 AM, Robert Munteanu wrote: > On Sat, 2016-01-09 at 14:48 +0100, Oliver Lietz wrote: > > Please vote to approve this release: > > +1 > > Robert

[jira] [Created] (SLING-5426) Expose size to distribution packages

2016-01-12 Thread Marius Petria (JIRA)
Marius Petria created SLING-5426: Summary: Expose size to distribution packages Key: SLING-5426 URL: https://issues.apache.org/jira/browse/SLING-5426 Project: Sling Issue Type: Improvement

[jira] [Commented] (SLING-5416) Thread Pool should stop "gracefully"

2016-01-12 Thread Thomas Mueller (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15093891#comment-15093891 ] Thomas Mueller commented on SLING-5416: --- In my view, calling Thread.interrupt is a bug, because it

[jira] [Resolved] (SLING-5426) Expose size to distribution packages

2016-01-12 Thread Marius Petria (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5426?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marius Petria resolved SLING-5426. -- Resolution: Fixed Committed revision 1724235. > Expose size to distribution packages >

[jira] [Commented] (SLING-5416) Thread Pool should stop "gracefully"

2016-01-12 Thread Thomas Mueller (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15093738#comment-15093738 ] Thomas Mueller commented on SLING-5416: --- > I guess the danger here is that the shutdown blocks

[jira] [Resolved] (SLING-5425) Set webconsole.configurationFactory.nameHint for thread pool configurations

2016-01-12 Thread Robert Munteanu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5425?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Munteanu resolved SLING-5425. Resolution: Fixed Fixed in https://svn.apache.org/r1724221 > Set

[jira] [Created] (SLING-5425) Set webconsole.configurationFactory.nameHint for thread pool configurations

2016-01-12 Thread Robert Munteanu (JIRA)
Robert Munteanu created SLING-5425: -- Summary: Set webconsole.configurationFactory.nameHint for thread pool configurations Key: SLING-5425 URL: https://issues.apache.org/jira/browse/SLING-5425

[jira] [Commented] (SLING-5416) Thread Pool should stop "gracefully"

2016-01-12 Thread Robert Munteanu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15093850#comment-15093850 ] Robert Munteanu commented on SLING-5416: That's a good point. However, as we don't ship a default

SLING-5416 - Change ThreadPool default configuration to not use Thread.interrupt

2016-01-12 Thread Robert Munteanu
Hi, There's an ongoing discussion at    https://issues.apache.org/jira/browse/SLING-5416 regarding whether the default ThreadPool configuration should allow usage of Thread.interrupt or not. Although I'm not 100% convinced that Thread.interrupt should be avoided, it looks like it's a better

[jira] [Commented] (SLING-5416) Thread Pool should stop "gracefully"

2016-01-12 Thread Robert Munteanu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15094017#comment-15094017 ] Robert Munteanu commented on SLING-5416: I've posted this for on [dev@sling - SLING-5416 - Change

[RESULT] [VOTE] Release Apache Sling Scripting Groovy 1.0.2

2016-01-12 Thread Oliver Lietz
Hi, the vote has passed with the following result: +1 (binding): Stefan Seifert, Robert Munteanu, Daniel Klco, Oliver Lietz I will copy this release to the Sling dist directory and promote the artifacts to the central Maven repository. Thanks for voting! O.

[jira] [Closed] (SLING-5378) Use version of Groovy in GSPScriptEngineFactory for language version

2016-01-12 Thread Oliver Lietz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5378?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Oliver Lietz closed SLING-5378. --- > Use version of Groovy in GSPScriptEngineFactory for language version >

[jira] [Closed] (SLING-5377) Update Groovy to 2.4.5

2016-01-12 Thread Oliver Lietz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5377?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Oliver Lietz closed SLING-5377. --- > Update Groovy to 2.4.5 > -- > > Key: SLING-5377 >

[jira] [Commented] (SLING-2752) Revisit re-saving of config settings in light of support for wider range of types.

2016-01-12 Thread JIRA
[ https://issues.apache.org/jira/browse/SLING-2752?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15094286#comment-15094286 ] Santiago García Pimentel commented on SLING-2752: - Just as a note for anyone looking into

[jira] [Comment Edited] (SLING-2752) Revisit re-saving of config settings in light of support for wider range of types.

2016-01-12 Thread JIRA
[ https://issues.apache.org/jira/browse/SLING-2752?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15094286#comment-15094286 ] Santiago García Pimentel edited comment on SLING-2752 at 1/12/16 5:00 PM:

[jira] [Created] (SLING-5427) The jcr:created and jcr:lastModified properties are not read for mocked resources created through the ContentLoader

2016-01-12 Thread Radu Cotescu (JIRA)
Radu Cotescu created SLING-5427: --- Summary: The jcr:created and jcr:lastModified properties are not read for mocked resources created through the ContentLoader Key: SLING-5427 URL: