Re: ThreadContextMap2

2016-08-23 Thread Gary Gregory
fect binary or source > compatibility, will it? > > On 23 August 2016 at 13:58, Gary Gregory wrote: > >> I made the change for ThreadContextMap2 to extend ThreadContextMap >> (yesterday). >> >> Gary >> >> On Mon, Aug 22, 2016 at 1:48 PM, Gary Gre

Re: ThreadContextMap2

2016-08-23 Thread Matt Sicker
I know we added TCM2 to preserve backwards compatibility. By making TCM2 extend TCM and the other changes, this won't affect binary or source compatibility, will it? On 23 August 2016 at 13:58, Gary Gregory wrote: > I made the change for ThreadContextMap2 to extend ThreadContextMap >

Re: ThreadContextMap2

2016-08-23 Thread Gary Gregory
I made the change for ThreadContextMap2 to extend ThreadContextMap (yesterday). Gary On Mon, Aug 22, 2016 at 1:48 PM, Gary Gregory wrote: > Why doesn't ThreadContextMap2 extend ThreadContextMap? This feels highly > unusual. > > Gary > > On Sat, Aug 20, 2016 at 5:10 PM

Re: ThreadContextMap2

2016-08-22 Thread Gary Gregory
Why doesn't ThreadContextMap2 extend ThreadContextMap? This feels highly unusual. Gary On Sat, Aug 20, 2016 at 5:10 PM, Gary Gregory wrote: > > On Aug 20, 2016 4:47 PM, "Remko Popma" wrote: > > > > Do you have anything specific in mind? > > Otherwise so

Re: ThreadContextMap2

2016-08-20 Thread Gary Gregory
> On Aug 20, 2016, at 11:01 AM, Gary Gregory wrote: >>>>> >>>>> Fair enough. >>>>> >>>>> I am wondering if there is a design reason not to implement Map now that we have the opportunity with the TCM2 interface. I'd take a crack at it. >>&g

Re: ThreadContextMap2

2016-08-20 Thread Remko Popma
crack at it. >> >> Gary >> >> On Aug 20, 2016 10:39 AM, "Ralph Goers" >> wrote: >> >>> Because Map has a lot more methods that weren’t needed and I didn’t want >>> to implement? >>> >>> Ralph >>> >>>

Re: ThreadContextMap2

2016-08-20 Thread Gary Gregory
t; Because Map has a lot more methods that weren’t needed and I didn’t want >> to implement? >> >> Ralph >> >> On Aug 20, 2016, at 10:16 AM, Gary Gregory >> wrote: >> >> Hi All: >> >> Now that we have ThreadContextMap2: >> >&

Re: ThreadContextMap2

2016-08-20 Thread Remko Popma
. I'd take a crack at it. >> >> Gary >> >> >>> On Aug 20, 2016 10:39 AM, "Ralph Goers" wrote: >>> Because Map has a lot more methods that weren’t needed and I didn’t want to >>> implement? >>> >>> Ralph >

Re: ThreadContextMap2

2016-08-20 Thread Ralph Goers
Because Map has a lot more methods that weren’t needed and I didn’t want to > implement? > > Ralph > >> On Aug 20, 2016, at 10:16 AM, Gary Gregory > <mailto:garydgreg...@gmail.com>> wrote: >> >> Hi All: >> >> Now that we have ThreadContextMap

Re: ThreadContextMap2

2016-08-20 Thread Gary Gregory
I didn’t want > to implement? > > Ralph > > On Aug 20, 2016, at 10:16 AM, Gary Gregory wrote: > > Hi All: > > Now that we have ThreadContextMap2: > > Why don't we do ThreadContextMap2 extends Map? > > Why did ThreadContextMap initially not extends

Re: ThreadContextMap2

2016-08-20 Thread Ralph Goers
Because Map has a lot more methods that weren’t needed and I didn’t want to implement? Ralph > On Aug 20, 2016, at 10:16 AM, Gary Gregory wrote: > > Hi All: > > Now that we have ThreadContextMap2: > > Why don't we do ThreadContextMap2 extends Map? > > Why di

ThreadContextMap2

2016-08-20 Thread Gary Gregory
Hi All: Now that we have ThreadContextMap2: Why don't we do ThreadContextMap2 extends Map? Why did ThreadContextMap initially not extends Map? Gary -- E-Mail: garydgreg...@gmail.com | ggreg...@apache.org Java Persistence with Hibernate, Second Edition <http://www.manning.com/bauer3