Re: [infinispan-dev] To Optional or not to Optional?

2017-05-25 Thread Sanne Grinovero
On 25 May 2017 at 09:00, Sebastian Laskawiec wrote: > Adding part of your email from REST refactoring thread: > > Looking into the code, without considering performance at all, I think >> that you've become too ecstatic about Optionals. These should be used as >> return

Re: [infinispan-dev] To Optional or not to Optional?

2017-05-25 Thread David M. Lloyd
I'm not an Infinispan developer, but I'll chime in anyway. :) I've never been a fan of Optional. But the theory behind it that made it acceptable in the first place is that it generally gets optimized away. Now this theory is only true if you never hold a reference to it in any persistent

Re: [infinispan-dev] To Optional or not to Optional?

2017-05-25 Thread Emmanuel Bernard
> On 25 May 2017, at 10:00, Sebastian Laskawiec wrote: > . As it turns out we can use Optionals anywhere, except method returning some > objects which are not getters. You can't use it on non getter return types ? Why ? > > It is also worth to say that both gentlemen

Re: [infinispan-dev] IRC chat: HB + I9

2017-05-25 Thread Steve Ebersole
A lot to read through here so I apologize up front if I missed something... So to be fair I am biased as I would really like to not have to deal with these integrations :) That said, I do really believe that the best option is to move this code out of the hibernate/hibernate-orm repo. To me

Re: [infinispan-dev] In Memory Data Grid Patterns Demos from Devoxx France!

2017-05-25 Thread Vojtech Juranek
On pondělí 22. května 2017 15:52:12 CEST Galder Zamarreño wrote: > Another thing, isn't the package.json file missign dependencies? yes, thanks for spotting it - it should be fixed now. I also initiated the transfer of the repo under infinispan-demos Cheers Vojta signature.asc Description:

Re: [infinispan-dev] REST Refactoring - breaking changes

2017-05-25 Thread Sebastian Laskawiec
On Thu, May 25, 2017 at 7:42 AM Radim Vansa wrote: > On 05/24/2017 10:44 AM, Sebastian Laskawiec wrote: > > > > > > On Tue, May 23, 2017 at 5:06 PM Radim Vansa > > wrote: > > > > On 05/16/2017 11:05 AM, Sebastian Laskawiec

Re: [infinispan-dev] To Optional or not to Optional?

2017-05-25 Thread Sebastian Laskawiec
Indeed Bela, you're an extreme naysayer! :) I'm actually trying to get as many comments and arguments out of this discussion. I hope we will be able to iron out a general recommendation or approach how we want to treat Optionals. On Tue, May 23, 2017 at 10:14 PM Bela Ban