[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-2d-dev
Subject:    Re: [OpenJDK 2D-Dev] JDK-8153732 would it be possible to extend the PrintServiceLookup with a refres
From:       Patrick Reinhart <patrick () reini ! net>
Date:       2016-05-25 21:18:03
Message-ID: B0D53E67-4A0D-46B5-BE9A-CF0914C39EE5 () reini ! net
[Download RAW message or body]

Hi Phil,

> Am 25.05.2016 um 22:28 schrieb Phil Race <philip.race@oracle.com>:
> 
> Possible, but should we ?

I think yes, but it should be clearly documented, that a call to this method can be \
expensive.

> That is the method that does the refresh once we've received the notification.
> We wanted to avoid applications having to continually call this method every time
> the applications asks for the list because it may be a lengthy operation.

In my opinion, this method should only be called in rare cases in order to force an \
update. That’s our current way to trigger an update without restarting the VM (using \
reflection)

> If it were not we probably would have just done that.
> 
> So then the app has to figure out a policy on how often to ask for a refresh
> and this is what we wanted to avoid.

It should state that this method is time consuming and should used with cause.

> And given that the reason to call the method is somewhat environmental
> it might be tricky to explain why and when to call it given that mostly it
> is not necessary.

That is true, on the other hand I see no better way to trigger such an update.

Cheers

Patrick


> -phil.
> 
> 
> 
> On 05/25/2016 12:59 PM, Patrick Reinhart wrote:
> > Hi everyone,
> > 
> > I know you that this issue does not have the highest priority. Even though I \
> > would like to work on a possible solution without having to relay exclusively on \
> > a OS specific solution. I have looked into the different actual implementations \
> > of the PrintServiceLookup and found out that there all having already such a \
> > functionality that is called refreshServices(). 
> > My question is now: Could this method not me made available as a public API \
> > method on the PrintServiceLookup? That would solve my problem in a pretty simple \
> > way, that if a remote printer is being changed this method could be called to \
> > manually refresh the printers. 
> > What is your opinion about this?
> > 
> > Cheers
> > 
> > Patrick
> 


["signature.asc" (signature.asc)]

-----BEGIN PGP SIGNATURE-----

iQEcBAEBCAAGBQJXRhaLAAoJEDmHbl4dVhgHevYH/39WiBF5qV8NHOpJcVAZBJT4
vxgmA0rNDcxeezmNrGocYsocYbMktsjV9cq2NcN3JOV/IqW5cA8F5tWOCRiFbtO/
TaIaf0CFyrP2Hccmk7Et2hFfVoKn29Aeyauj8WTvfOp3pKiQFkhppcE6ed99jFFE
H7Khn1CmzdrRIj45WYtZNHbndYsEfMVd2S39jiSqWJ4xr/xrl5vG2RovLN8ya/dX
eRbqxxY1Ijd5irP0g56kvk+YvB0zeRUw2we9hEkn0F38cl4yNagKuSNxJJpyatfq
FFvhZXzG+RSznNxCFMYXJBE3GixaRpnYavnp24QaeMYLf64LO3/YGvuys5AYNNc=
=A94W
-----END PGP SIGNATURE-----


[prev in list] [next in list] [prev in thread] [next in thread]