[GitHub] karaf pull request #369: [KARAF-5375] feature:stop command does not stop its...

2017-09-14 Thread gnodet
GitHub user gnodet opened a pull request: https://github.com/apache/karaf/pull/369 [KARAF-5375] feature:stop command does not stop its bundles You can merge this pull request into a Git repository by running: $ git pull https://github.com/gnodet/karaf KARAF-5375

[GitHub] karaf pull request #368: Speed up string splitting operations

2017-09-14 Thread rovarga
GitHub user rovarga opened a pull request: https://github.com/apache/karaf/pull/368 Speed up string splitting operations Instead of performing multiple String.indexOf(String) operations, use String.indexOf(char) and cache its result. Signed-off-by: Robert Varga

[GitHub] karaf pull request #367: Use Map.computeIfAbsent() in map population

2017-09-14 Thread rovarga
GitHub user rovarga opened a pull request: https://github.com/apache/karaf/pull/367 Use Map.computeIfAbsent() in map population Instead of performing two lookups, perform a computeIfAbsent(), which perform a fused lookup-or-put. Signed-off-by: Robert Varga

[GitHub] karaf pull request #366: Speed up repository loading

2017-09-14 Thread rovarga
Github user rovarga closed the pull request at: https://github.com/apache/karaf/pull/366 ---

[GitHub] karaf pull request #366: Speed up repository loading

2017-09-14 Thread rovarga
GitHub user rovarga opened a pull request: https://github.com/apache/karaf/pull/366 Speed up repository loading toLoad is used as a java.util.Queue, but is backed by an ArrayList, which is highly inefficient when large number of items are present. Turn it into a Queue

Re: "Dynamic" org.apache.karaf.features.internal.service.Overrides#override()

2017-09-14 Thread Grzegorz Grzybek
I've created https://issues.apache.org/jira/browse/KARAF-5376 to track this regards Grzegorz Grzybek 2017-09-14 11:58 GMT+02:00 Grzegorz Grzybek : > Thanks for answer > > 2017-09-14 11:40 GMT+02:00 Guillaume Nodet : >> >> 2017-09-14 9:59 GMT+02:00

Re: "Dynamic" org.apache.karaf.features.internal.service.Overrides#override()

2017-09-14 Thread Grzegorz Grzybek
Thanks for answer 2017-09-14 11:40 GMT+02:00 Guillaume Nodet : > > 2017-09-14 9:59 GMT+02:00 Grzegorz Grzybek : > > > By default we can use etc/overrides.properties file to instruct Karaf to > > install e.g., mvn:commons-beanutils/commons-beanutils/1.9.3

Re: "Dynamic" org.apache.karaf.features.internal.service.Overrides#override()

2017-09-14 Thread Guillaume Nodet
2017-09-14 9:59 GMT+02:00 Grzegorz Grzybek : > Hello > > I'd like to discuss an idea. Currently *only* > org.apache.karaf.features.internal.service.Overrides#shouldOverride() > method decides whether (during feature installation/provisioning) given > bundle/resource URI from

"Dynamic" org.apache.karaf.features.internal.service.Overrides#override()

2017-09-14 Thread Grzegorz Grzybek
Hello I'd like to discuss an idea. Currently *only* org.apache.karaf.features.internal.service.Overrides#shouldOverride() method decides whether (during feature installation/provisioning) given bundle/resource URI from a feature should be "overriden" by some slightly different URI. By default we