Re: [lang] org.apache.commons.lang3.concurrent.Locks.Lock

2020-07-10 Thread Gary Gregory
On Wed, Jul 8, 2020 at 10:13 AM Gary Gregory wrote: > > > On Wed, Jul 8, 2020, 09:32 Rob Tompkins wrote: > >> >> >> > On Jul 7, 2020, at 6:56 PM, Gary Gregory >> wrote: >> > >> > In the PR https://github.com/apache/commons-lang/pull/559 I am going >> with >> > "LockingVistors". >> >> I like th

Re: [lang] org.apache.commons.lang3.concurrent.Locks.Lock

2020-07-08 Thread Gary Gregory
On Wed, Jul 8, 2020, 09:32 Rob Tompkins wrote: > > > > On Jul 7, 2020, at 6:56 PM, Gary Gregory wrote: > > > > In the PR https://github.com/apache/commons-lang/pull/559 I am going > with > > "LockingVistors". > > I like this name because of its brevity yet clarity. > Let's go for that then. I'

Re: [lang] org.apache.commons.lang3.concurrent.Locks.Lock

2020-07-08 Thread Rob Tompkins
> On Jul 7, 2020, at 6:56 PM, Gary Gregory wrote: > > In the PR https://github.com/apache/commons-lang/pull/559 I am going with > "LockingVistors". I like this name because of its brevity yet clarity. -Rob > > Gary > > On Tue, Jul 7, 2020 at 2:33 PM Rob Tompkins wrote: > >> I’m not all

Re: [lang] org.apache.commons.lang3.concurrent.Locks.Lock

2020-07-08 Thread Gary Gregory
On Tue, Jul 7, 2020 at 10:22 PM Gary Gregory wrote: > On Tue, Jul 7, 2020 at 7:14 PM Matt Sicker wrote: > >> JCIP seems to call this idea a monitor, but that’s also the general >> implicit locking mechanism in Java. >> > FTR, that's JCIP section 4.2.1 (in my edition). Gary > > Hi Matt, > > Ri

Re: [lang] org.apache.commons.lang3.concurrent.Locks.Lock

2020-07-07 Thread Gary Gregory
On Tue, Jul 7, 2020 at 7:14 PM Matt Sicker wrote: > JCIP seems to call this idea a monitor, but that’s also the general > implicit locking mechanism in Java. > Hi Matt, Right, and it is tempting to rename LockingVisitors.AbstractLockVisitor's 'object' ivar to 'monitor' but that is somewhat misl

Re: [lang] org.apache.commons.lang3.concurrent.Locks.Lock

2020-07-07 Thread Matt Sicker
JCIP seems to call this idea a monitor, but that’s also the general implicit locking mechanism in Java. On Tue, Jul 7, 2020 at 17:56 Gary Gregory wrote: > Typo: LockingVisitors. > > On Tue, Jul 7, 2020 at 6:56 PM Gary Gregory > wrote: > > > In the PR https://github.com/apache/commons-lang/pull

Re: [lang] org.apache.commons.lang3.concurrent.Locks.Lock

2020-07-07 Thread Gary Gregory
Typo: LockingVisitors. On Tue, Jul 7, 2020 at 6:56 PM Gary Gregory wrote: > In the PR https://github.com/apache/commons-lang/pull/559 I am going > with "LockingVistors". > > Gary > > On Tue, Jul 7, 2020 at 2:33 PM Rob Tompkins wrote: > >> I’m not all that familiar with that area of academia. L

Re: [lang] org.apache.commons.lang3.concurrent.Locks.Lock

2020-07-07 Thread Gary Gregory
In the PR https://github.com/apache/commons-lang/pull/559 I am going with "LockingVistors". Gary On Tue, Jul 7, 2020 at 2:33 PM Rob Tompkins wrote: > I’m not all that familiar with that area of academia. Let me see what I > can dig up > > -Rob > > > On Jul 7, 2020, at 2:19 PM, Matt Sicker wro

Re: [lang] org.apache.commons.lang3.concurrent.Locks.Lock

2020-07-07 Thread Rob Tompkins
I’m not all that familiar with that area of academia. Let me see what I can dig up -Rob > On Jul 7, 2020, at 2:19 PM, Matt Sicker wrote: > > Are there any academic references for this? Or even something from > Java Concurrency in Practice? Those are good sources for names around > concurrency.

Re: [lang] org.apache.commons.lang3.concurrent.Locks.Lock

2020-07-07 Thread Matt Sicker
Are there any academic references for this? Or even something from Java Concurrency in Practice? Those are good sources for names around concurrency. On Tue, 7 Jul 2020 at 12:42, Rob Tompkins wrote: > > Why not make the name “ThreadedLambdaSynchronizer” or something like > that….just something m

Re: [lang] org.apache.commons.lang3.concurrent.Locks.Lock

2020-07-07 Thread Rob Tompkins
Why not make the name “ThreadedLambdaSynchronizer” or something like that….just something more descriptive here that get’s closer to what the intended functionality is. That said, I’m not particularly tied to the name “ThreadedLambdaSynchronizer” just spitballing here for something more descrip

Re: [lang] org.apache.commons.lang3.concurrent.Locks.Lock

2020-06-29 Thread Matt Sicker
Now that starts to sound like Apache Groovy or Kotlin. On Mon, 29 Jun 2020 at 11:58, Xeno Amess wrote: > > soemtimes I really wish to rewrite/add some functions in jdk directly... > especially for reusing some package private static functions... > > Gary Gregory 于2020年6月30日周二 上午12:01写道: > > > I'

Re: [lang] org.apache.commons.lang3.concurrent.Locks.Lock

2020-06-29 Thread Xeno Amess
soemtimes I really wish to rewrite/add some functions in jdk directly... especially for reusing some package private static functions... Gary Gregory 于2020年6月30日周二 上午12:01写道: > I'm not sure talking to the JDK folks is helpful IMO. We are still > targeting Java 8. The customers I deal with are mi

Re: [lang] org.apache.commons.lang3.concurrent.Locks.Lock

2020-06-29 Thread Gary Gregory
I'm not sure talking to the JDK folks is helpful IMO. We are still targeting Java 8. The customers I deal with are migrating from 8 to 11, and Java 11 is not everywhere our customers are. So talking about something that might end up in Java... 25 seems to be not in our user's best or immediate inte

Re: [lang] org.apache.commons.lang3.concurrent.Locks.Lock

2020-06-29 Thread Rob Tompkins
At first look, I’m a little surprised we’re trying to take on this functionality. Has anyone reached out to the JDK guys to see if they’d be interested in having it in the JDK? That said, if we approach it from that path, we would lose the functionality in older versions of java. So maybe I jus

Re: [lang] org.apache.commons.lang3.concurrent.Locks.Lock

2020-06-26 Thread Gary Gregory
The 'Locks' class is in master. My PR is in my 1st message. Gary On Fri, Jun 26, 2020 at 5:27 PM Miguel Muñoz wrote: > The "lockedObject" might be better named "lockableObject." That way it > makes no assumptions about its current state. Building on this, the class > might be called Lockable. >

Re: [lang] org.apache.commons.lang3.concurrent.Locks.Lock

2020-06-26 Thread Miguel Muñoz
The "lockedObject" might be better named "lockableObject." That way it makes no assumptions about its current state. Building on this, the class might be called Lockable. However, I also like the idea of calling the class Locker, as in a gym locker. The idea is that it holds something important. O

[lang] org.apache.commons.lang3.concurrent.Locks.Lock

2020-06-26 Thread Gary Gregory
Hi All: I know email is a challenging medium for code reviews, so please consider these comments coming from my best intentions, constructive and caring ;-) Also please excuse the meandering nature of this post. The new class org.apache.commons.lang3.concurrent.Locks.Lock needs better names IMO,