Mailing-List: contact dev-help@polygene.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@polygene.apache.org
MIME-Version: 1.0
Sender: hedhman@gmail.com
In-Reply-To: <58E909E9.5080209@apache.org>
References: <58E909E9.5080209@apache.org>
From: Niclas Hedhman <niclas@hedhman.org>
Date: Sun, 9 Apr 2017 07:49:21 +0800
Message-ID: <CADmm+KdDxz62XZpW4-Lo6CwqnsMP2mU6RLtOHXc=MDmbirB4Ag@mail.gmail.com>
Subject: Re: setAccessible( true )
To: dev@polygene.apache.org
Content-Type: multipart/alternative; boundary=f403045f7fc8607cfa054cb06558
archived-at: Sat, 08 Apr 2017 23:49:47 -0000

--f403045f7fc8607cfa054cb06558
Content-Type: text/plain; charset=UTF-8

Yeah, already pre-1.0 we were fighting to move all setAccessible() to model
creation where they belong and ensure none were executed in runtime. IF
they have snuck back into runtime, then yes they should be pushed back. IF
they are simply called too many times in bootstrap, then I don't worry too
much about it and wouldn't spend time locate them.

Cheers
Niclas

On Sun, Apr 9, 2017 at 12:03 AM, Paul Merlin <paulmerlin@apache.org> wrote:

> Gang,
>
> We use accessibleObject.setAccessible( true ) a bit too agressively.
> This is a performance killer because of the JVM security checks it implies.
> Some profiling showed me that's it's often on the critical path, Well
> that's because I profiled the bootstrap phase for most of the cases.
>
> BTW, the new serialization subsystem is way faster than the previous one
> \o/
> JSONEntityState still is a hotspot, but that's another topic.
>
> There are places where setAccessible( true ) is simply not needed and I
> will remove them.
>
> Unfortunately, o.isAccessible() does not check if it is actually
> accessible but simply if setAccessible( true ) was called on it to
> forcibly make it accessible. We have some of these checks here and
> there, I'll verify if they are legitimate.
>
> There are places where it supports e.g. using private classes as mixins,
> and injection to private fields. For some types of
> composites/fragments/etc.. calls to setAccessible() are made when
> creating the model, for some others they are made during runtime
> repeatedly. I'll move all these at bootstrap time so we don't pay that
> cost at runtime.
>
> /Paul
>
>


-- 
Niclas Hedhman, Software Developer
http://polygene.apache.org - New Energy for Java

--f403045f7fc8607cfa054cb06558--