On 12/3/2014 8:18 AM, Lance Andersen wrote:
Thank you Sean. As this code path is only called 1 time, i am not
concerned that performance will be an issue. If you and Mandy prefer
me to remove it, I can, just let me know.
Yes, I agree it is narrow. The suggestion to add the limited
doPriviliged came up in a review of RowSetProvider which is why I
figured I would add it here also. Now that was quite some time ago,
so I understand our position might have changed.
Since Sean will look into the guide when it's best to use limited
doPrivileged, you can add the appropriate ones later in JDK 9 and taking
it out also makes it easier to backport if any customer requests it.
I'm okay with webrev.04 and no need to generate a new webrev with the
doPrivileged change reverted.
Thanks
Mandy
