You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Felix Meschberger (JIRA)" <ji...@apache.org> on 2011/01/10 19:19:45 UTC

[jira] Created: (SLING-1925) Add sun.reflect to the default org.osgi.framework.bootdelegation system property

Add sun.reflect to the default org.osgi.framework.bootdelegation system property
--------------------------------------------------------------------------------

                 Key: SLING-1925
                 URL: https://issues.apache.org/jira/browse/SLING-1925
             Project: Sling
          Issue Type: Improvement
          Components: Launchpad
    Affects Versions:  Launchpad Base 2.2.0
            Reporter: Felix Meschberger
            Assignee: Felix Meschberger
             Fix For: Launchpad Base 2.2.2


There are known issues with recent Felix Framework versions around loading classes from the sun.reflect package to support Hotspot optimizations. See [1] for example.

To easily fix these problems, we should add this package to the default setup of the org.osgi.framework.bootdelegation system property. This helps a lot on Sun JVMs but does not hurt at all on non-Sun JVMs.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SLING-1925) Add sun.reflect to the default org.osgi.framework.bootdelegation system property

Posted by "Michael Dürig (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SLING-1925?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12980054#action_12980054 ] 

Michael Dürig commented on SLING-1925:
--------------------------------------

While this works and also seems to fix SLING-1895, I think adding an implementation dependency to the sun JVM is not correct. IMO the correct way would be to use a JVM specific fragment bundle for adding such additional boot delegation entries and attach it to the system bundle.


> Add sun.reflect to the default org.osgi.framework.bootdelegation system property
> --------------------------------------------------------------------------------
>
>                 Key: SLING-1925
>                 URL: https://issues.apache.org/jira/browse/SLING-1925
>             Project: Sling
>          Issue Type: Improvement
>          Components: Launchpad
>    Affects Versions:  Launchpad Base 2.2.0
>            Reporter: Felix Meschberger
>            Assignee: Felix Meschberger
>             Fix For: Launchpad Base 2.2.2
>
>
> There are known issues with recent Felix Framework versions around loading classes from the sun.reflect package to support Hotspot optimizations. See [1] for example.
> To easily fix these problems, we should add this package to the default setup of the org.osgi.framework.bootdelegation system property. This helps a lot on Sun JVMs but does not hurt at all on non-Sun JVMs.
> [1] http://www.mail-archive.com/users@felix.apache.org/msg07721.html

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SLING-1925) Add sun.reflect to the default org.osgi.framework.bootdelegation system property

Posted by "Felix Meschberger (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SLING-1925?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12980130#action_12980130 ] 

Felix Meschberger commented on SLING-1925:
------------------------------------------

I tried to cover this aspect in my initial node: Basically this is just a door opener for some internal HotSpot code which happens to exist on Sun/Oracle JVMs. I don't expect other code to ever call something in the sun.reflect package unless it is HotSpot code derived from the Sun/Oracle JVMs. Code never needing this package is not affected.

So, at the end of the day it is not an implementation dependency.

Creating a VM specific fragment bundle would be another option. But that would mean almost all users of Sling would have to install this bundle. Over the last few weeks, I came to the conclusion, that the purity of setup is not worth the price in this particular context. Consider it a special case, I do.

> Add sun.reflect to the default org.osgi.framework.bootdelegation system property
> --------------------------------------------------------------------------------
>
>                 Key: SLING-1925
>                 URL: https://issues.apache.org/jira/browse/SLING-1925
>             Project: Sling
>          Issue Type: Improvement
>          Components: Launchpad
>    Affects Versions:  Launchpad Base 2.2.0
>            Reporter: Felix Meschberger
>            Assignee: Felix Meschberger
>             Fix For: Launchpad Base 2.2.2
>
>
> There are known issues with recent Felix Framework versions around loading classes from the sun.reflect package to support Hotspot optimizations. See [1] for example.
> To easily fix these problems, we should add this package to the default setup of the org.osgi.framework.bootdelegation system property. This helps a lot on Sun JVMs but does not hurt at all on non-Sun JVMs.
> [1] http://www.mail-archive.com/users@felix.apache.org/msg07721.html

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SLING-1925) Add sun.reflect to the default org.osgi.framework.bootdelegation system property

Posted by "Felix Meschberger (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SLING-1925?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Felix Meschberger updated SLING-1925:
-------------------------------------

    Description: 
There are known issues with recent Felix Framework versions around loading classes from the sun.reflect package to support Hotspot optimizations. See [1] for example.

To easily fix these problems, we should add this package to the default setup of the org.osgi.framework.bootdelegation system property. This helps a lot on Sun JVMs but does not hurt at all on non-Sun JVMs.

[1] http://www.mail-archive.com/users@felix.apache.org/msg07721.html

  was:
There are known issues with recent Felix Framework versions around loading classes from the sun.reflect package to support Hotspot optimizations. See [1] for example.

To easily fix these problems, we should add this package to the default setup of the org.osgi.framework.bootdelegation system property. This helps a lot on Sun JVMs but does not hurt at all on non-Sun JVMs.


> Add sun.reflect to the default org.osgi.framework.bootdelegation system property
> --------------------------------------------------------------------------------
>
>                 Key: SLING-1925
>                 URL: https://issues.apache.org/jira/browse/SLING-1925
>             Project: Sling
>          Issue Type: Improvement
>          Components: Launchpad
>    Affects Versions:  Launchpad Base 2.2.0
>            Reporter: Felix Meschberger
>            Assignee: Felix Meschberger
>             Fix For: Launchpad Base 2.2.2
>
>
> There are known issues with recent Felix Framework versions around loading classes from the sun.reflect package to support Hotspot optimizations. See [1] for example.
> To easily fix these problems, we should add this package to the default setup of the org.osgi.framework.bootdelegation system property. This helps a lot on Sun JVMs but does not hurt at all on non-Sun JVMs.
> [1] http://www.mail-archive.com/users@felix.apache.org/msg07721.html

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (SLING-1925) Add sun.reflect to the default org.osgi.framework.bootdelegation system property

Posted by "Felix Meschberger (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SLING-1925?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Felix Meschberger resolved SLING-1925.
--------------------------------------

    Resolution: Fixed

Fixed in Rev. 1069719 by adding a new property:

    sling.bootdelegation.sun.reflect = sun.reflect

This will just be added to any existing sling.proprties file (in an upgrade scenario) and be added to the actual org.osgi.framework.bootdelegation system property by the Sling launcher

> Add sun.reflect to the default org.osgi.framework.bootdelegation system property
> --------------------------------------------------------------------------------
>
>                 Key: SLING-1925
>                 URL: https://issues.apache.org/jira/browse/SLING-1925
>             Project: Sling
>          Issue Type: Improvement
>          Components: Launchpad
>    Affects Versions:  Launchpad Base 2.2.0
>            Reporter: Felix Meschberger
>            Assignee: Felix Meschberger
>             Fix For: Launchpad Base 2.2.2
>
>
> There are known issues with recent Felix Framework versions around loading classes from the sun.reflect package to support Hotspot optimizations. See [1] for example.
> To easily fix these problems, we should add this package to the default setup of the org.osgi.framework.bootdelegation system property. This helps a lot on Sun JVMs but does not hurt at all on non-Sun JVMs.
> [1] http://www.mail-archive.com/users@felix.apache.org/msg07721.html

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Issue Comment Edited: (SLING-1925) Add sun.reflect to the default org.osgi.framework.bootdelegation system property

Posted by "Felix Meschberger (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SLING-1925?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12980130#comment-12980130 ] 

Felix Meschberger edited comment on SLING-1925 at 2/10/11 3:42 PM:
-------------------------------------------------------------------

I tried to cover this aspect in my initial note: Basically this is just a door opener for some internal HotSpot code which happens to exist on Sun/Oracle JVMs. I don't expect other code to ever call something in the sun.reflect package unless it is HotSpot code derived from the Sun/Oracle JVMs. Code never needing this package is not affected.

So, at the end of the day it is not an implementation dependency.

Creating a VM specific fragment bundle would be another option. But that would mean almost all users of Sling would have to install this bundle. Over the last few weeks, I came to the conclusion, that the purity of setup is not worth the price in this particular context. Consider it a special case, I do.

      was (Author: fmeschbe):
    I tried to cover this aspect in my initial node: Basically this is just a door opener for some internal HotSpot code which happens to exist on Sun/Oracle JVMs. I don't expect other code to ever call something in the sun.reflect package unless it is HotSpot code derived from the Sun/Oracle JVMs. Code never needing this package is not affected.

So, at the end of the day it is not an implementation dependency.

Creating a VM specific fragment bundle would be another option. But that would mean almost all users of Sling would have to install this bundle. Over the last few weeks, I came to the conclusion, that the purity of setup is not worth the price in this particular context. Consider it a special case, I do.
  
> Add sun.reflect to the default org.osgi.framework.bootdelegation system property
> --------------------------------------------------------------------------------
>
>                 Key: SLING-1925
>                 URL: https://issues.apache.org/jira/browse/SLING-1925
>             Project: Sling
>          Issue Type: Improvement
>          Components: Launchpad
>    Affects Versions:  Launchpad Base 2.2.0
>            Reporter: Felix Meschberger
>            Assignee: Felix Meschberger
>             Fix For: Launchpad Base 2.2.2
>
>
> There are known issues with recent Felix Framework versions around loading classes from the sun.reflect package to support Hotspot optimizations. See [1] for example.
> To easily fix these problems, we should add this package to the default setup of the org.osgi.framework.bootdelegation system property. This helps a lot on Sun JVMs but does not hurt at all on non-Sun JVMs.
> [1] http://www.mail-archive.com/users@felix.apache.org/msg07721.html

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Resolved: (SLING-1925) Add sun.reflect to the default org.osgi.framework.bootdelegation system property

Posted by "Felix Meschberger (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SLING-1925?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Felix Meschberger resolved SLING-1925.
--------------------------------------

    Resolution: Fixed

Fixed in Rev. 1057296.

> Add sun.reflect to the default org.osgi.framework.bootdelegation system property
> --------------------------------------------------------------------------------
>
>                 Key: SLING-1925
>                 URL: https://issues.apache.org/jira/browse/SLING-1925
>             Project: Sling
>          Issue Type: Improvement
>          Components: Launchpad
>    Affects Versions:  Launchpad Base 2.2.0
>            Reporter: Felix Meschberger
>            Assignee: Felix Meschberger
>             Fix For: Launchpad Base 2.2.2
>
>
> There are known issues with recent Felix Framework versions around loading classes from the sun.reflect package to support Hotspot optimizations. See [1] for example.
> To easily fix these problems, we should add this package to the default setup of the org.osgi.framework.bootdelegation system property. This helps a lot on Sun JVMs but does not hurt at all on non-Sun JVMs.
> [1] http://www.mail-archive.com/users@felix.apache.org/msg07721.html

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Reopened: (SLING-1925) Add sun.reflect to the default org.osgi.framework.bootdelegation system property

Posted by "Felix Meschberger (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SLING-1925?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Felix Meschberger reopened SLING-1925:
--------------------------------------


The fix of Rev. 1057296 has an issue in update/upgrade scenarios: While it works perfectly in fresh installations, when upgrading the launchpad base it may not be picked up because the org.osgi.framework.bootdelegation property of an existing sling.properties file is not overwritten.

> Add sun.reflect to the default org.osgi.framework.bootdelegation system property
> --------------------------------------------------------------------------------
>
>                 Key: SLING-1925
>                 URL: https://issues.apache.org/jira/browse/SLING-1925
>             Project: Sling
>          Issue Type: Improvement
>          Components: Launchpad
>    Affects Versions:  Launchpad Base 2.2.0
>            Reporter: Felix Meschberger
>            Assignee: Felix Meschberger
>             Fix For: Launchpad Base 2.2.2
>
>
> There are known issues with recent Felix Framework versions around loading classes from the sun.reflect package to support Hotspot optimizations. See [1] for example.
> To easily fix these problems, we should add this package to the default setup of the org.osgi.framework.bootdelegation system property. This helps a lot on Sun JVMs but does not hurt at all on non-Sun JVMs.
> [1] http://www.mail-archive.com/users@felix.apache.org/msg07721.html

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira