Re: [hibernate-dev] HHH-3138 - make constructor of DynamicMapEntityTuplizer public

2018-07-17 Thread Steve Ebersole
Sorry for the late reply. As Sanne indicated, we are all very busy with our own work. Reaching out here and "championing" a particular issue that you are interested in working on is a great way to make sure it is not missed. Anyway, I replied on the Jira. On Tue, Jul 17, 2018 at 8:11 AM Sanne G

Re: [hibernate-dev] HHH-3138 - make constructor of DynamicMapEntityTuplizer public

2018-07-17 Thread Sanne Grinovero
On Tue, 17 Jul 2018 at 13:45, Guillaume Smet wrote: > > Hi, > > On Tue, Jul 17, 2018 at 11:33 AM Sanne Grinovero wrote: >> >> In the specific example of HHH-3138, I guess people didn't comment as >> there is no clear problem to be solved. It might help to explain why >> you want to subclass a Dyn

Re: [hibernate-dev] HHH-3138 - make constructor of DynamicMapEntityTuplizer public

2018-07-17 Thread Guillaume Smet
Hi, On Tue, Jul 17, 2018 at 11:33 AM Sanne Grinovero wrote: > In the specific example of HHH-3138, I guess people didn't comment as > there is no clear problem to be solved. It might help to explain why > you want to subclass a DynamicMapEntityTuplizer - but I'm just > guessing as I'm not famili

Re: [hibernate-dev] HHH-3138 - make constructor of DynamicMapEntityTuplizer public

2018-07-17 Thread Sanne Grinovero
Hi Alvaro, welcome :-) The ticket descriptions can be debated; we can all be wrong, or you might have a better idea. Regarding feedback times: I'm sorry we're a bit slow. There are many things which are urgent - or we're more interested in - and since it's summer so many are on holidays. It migh

[hibernate-dev] HHH-3138 - make constructor of DynamicMapEntityTuplizer public

2018-07-16 Thread Alvaro Pedraza
Hi, I asigned myself this ticket and left a comment but unfortunately no feedback yet. https://hibernate.atlassian.net/browse/HHH-3138 This leads me to 2 questions: The first is the very same in the comment, which is if subclassing in DynamicMapEntityTuplizer is desirable? Please refer to the tic