You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by joelrich <gi...@git.apache.org> on 2015/10/07 12:22:02 UTC

[GitHub] sling pull request: SLING-5108 - Various little performance improv...

GitHub user joelrich opened a pull request:

    https://github.com/apache/sling/pull/105

    SLING-5108 - Various little performance improvements

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/joelrich/sling SLING-5108

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/sling/pull/105.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #105
    
----
commit 1a2c72a1ec89f12a23f953d50959756fedb77f60
Author: Joel Richard <jo...@adobe.com>
Date:   2015-10-07T09:46:52Z

    SLING-5108 - Various little performance improvements
    
    * Do not call hasProperty twice if the key is the same. In Oak, even if the property is cached, hasProperty is not for free because it performs a session operation with pre and post tasks.

commit 3bd06593a3d1d1e17454042a997a74f9505bea8c
Author: Joel Richard <jo...@adobe.com>
Date:   2015-10-07T09:55:46Z

    SLING-5108 - Various little performance improvements
    
    * Do not use String.format in getPath because this method is called very often

commit 74fd994d7a2cebe8bd7abcb409e4d42cc41a665a
Author: Joel Richard <jo...@adobe.com>
Date:   2015-10-07T09:45:19Z

    SLING-5108 - Various little performance improvements
    
    * Do not call getSimpleName() twice (because it is not just a cheap getter)

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] sling pull request: SLING-5108 - Various little performance improv...

Posted by joelrich <gi...@git.apache.org>.
Github user joelrich closed the pull request at:

    https://github.com/apache/sling/pull/105


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---