Re: [DISCUSSION] Back porting: Remove all the fk dependencies from stats entities to the business entities

2018-12-22 Thread Aditya Sharma
Thanks Deepak :)

With no conflicts yet done at r1849541 for R16.11.

Thanks and Regards,

*Aditya Sharma* | Enterprise Software Engineer
HotWax Commerce  by HotWax Systems

[image: https://www.linkedin.com/in/aditya-p-sharma/]



On Wed, Dec 19, 2018 at 4:15 PM Deepak Dixit  wrote:

> +1 we can backport these changes to R16.11.
>
>
> Thanks & Regards
> --
> Deepak Dixit
>
>
>
> On Wed, Dec 19, 2018 at 1:41 PM Aditya Sharma <
> aditya.sha...@hotwaxsystems.com> wrote:
>
> > Hello everyone,
> >
> > As described on OFBIZ-10721
> > 
> >
> > As per the previous discussion
> >  on dev ml and OFBIZ-9356
> > ,  stats related
> > entities
> > foreign key relation has been removed by Jacopo at r#1647271 and new
> group
> > org.apache.ofbiz.stats introduced for stats related entity.  I found some
> > remnant still left in stats entities.
> >
> > Changes that are done at r1647271 exist in Release 16 too, in my opinion,
> > backporting it won't harm as there are no foreign keys defined at the
> > database level as the database are different.
> >
> > WDYT?
> >
> > Thanks and Regards,
> >
> > *Aditya Sharma* | Enterprise Software Engineer
> > HotWax Commerce  by HotWax Systems
> > 
> > [image: https://www.linkedin.com/in/aditya-p-sharma/]
> > 
> >
>


Re: [DISCUSSION] Back porting: Remove all the fk dependencies from stats entities to the business entities

2018-12-19 Thread Deepak Dixit
+1 we can backport these changes to R16.11.


Thanks & Regards
--
Deepak Dixit



On Wed, Dec 19, 2018 at 1:41 PM Aditya Sharma <
aditya.sha...@hotwaxsystems.com> wrote:

> Hello everyone,
>
> As described on OFBIZ-10721
> 
>
> As per the previous discussion
>  on dev ml and OFBIZ-9356
> ,  stats related
> entities
> foreign key relation has been removed by Jacopo at r#1647271 and new group
> org.apache.ofbiz.stats introduced for stats related entity.  I found some
> remnant still left in stats entities.
>
> Changes that are done at r1647271 exist in Release 16 too, in my opinion,
> backporting it won't harm as there are no foreign keys defined at the
> database level as the database are different.
>
> WDYT?
>
> Thanks and Regards,
>
> *Aditya Sharma* | Enterprise Software Engineer
> HotWax Commerce  by HotWax Systems
> 
> [image: https://www.linkedin.com/in/aditya-p-sharma/]
> 
>


[DISCUSSION] Back porting: Remove all the fk dependencies from stats entities to the business entities

2018-12-19 Thread Aditya Sharma
Hello everyone,

As described on OFBIZ-10721


As per the previous discussion
 on dev ml and OFBIZ-9356
,  stats related entities
foreign key relation has been removed by Jacopo at r#1647271 and new group
org.apache.ofbiz.stats introduced for stats related entity.  I found some
remnant still left in stats entities.

Changes that are done at r1647271 exist in Release 16 too, in my opinion,
backporting it won't harm as there are no foreign keys defined at the
database level as the database are different.

WDYT?

Thanks and Regards,

*Aditya Sharma* | Enterprise Software Engineer
HotWax Commerce  by HotWax Systems

[image: https://www.linkedin.com/in/aditya-p-sharma/]