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

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

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

Aditya Sharma
Hello everyone,

As described on OFBIZ-10721
<https://issues.apache.org/jira/browse/OFBIZ-10721>

As per the previous discussion
<https://markmail.org/message/itdzyzfv7jwdr5jj> on dev ml and OFBIZ-9356
<https://issues.apache.org/jira/browse/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 <http://www.hotwax.co/> by HotWax Systems
<http://www.hotwaxsystems.com/>
[image: https://www.linkedin.com/in/aditya-p-sharma/]
<https://www.linkedin.com/in/aditya-p-sharma/>
Reply | Threaded
Open this post in threaded view
|

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

Deepak Dixit-4
+1 we can backport these changes to R16.11.


Thanks & Regards
--
Deepak Dixit



On Wed, Dec 19, 2018 at 1:41 PM Aditya Sharma <
[hidden email]> wrote:

> Hello everyone,
>
> As described on OFBIZ-10721
> <https://issues.apache.org/jira/browse/OFBIZ-10721>
>
> As per the previous discussion
> <https://markmail.org/message/itdzyzfv7jwdr5jj> on dev ml and OFBIZ-9356
> <https://issues.apache.org/jira/browse/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 <http://www.hotwax.co/> by HotWax Systems
> <http://www.hotwaxsystems.com/>
> [image: https://www.linkedin.com/in/aditya-p-sharma/]
> <https://www.linkedin.com/in/aditya-p-sharma/>
>
Reply | Threaded
Open this post in threaded view
|

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

Aditya Sharma
Thanks Deepak :)

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

Thanks and Regards,

*Aditya Sharma* | Enterprise Software Engineer
HotWax Commerce <http://www.hotwax.co/> by HotWax Systems
<http://www.hotwaxsystems.com/>
[image: https://www.linkedin.com/in/aditya-p-sharma/]
<https://www.linkedin.com/in/aditya-p-sharma/>


On Wed, Dec 19, 2018 at 4:15 PM Deepak Dixit <[hidden email]> 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 <
> [hidden email]> wrote:
>
> > Hello everyone,
> >
> > As described on OFBIZ-10721
> > <https://issues.apache.org/jira/browse/OFBIZ-10721>
> >
> > As per the previous discussion
> > <https://markmail.org/message/itdzyzfv7jwdr5jj> on dev ml and OFBIZ-9356
> > <https://issues.apache.org/jira/browse/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 <http://www.hotwax.co/> by HotWax Systems
> > <http://www.hotwaxsystems.com/>
> > [image: https://www.linkedin.com/in/aditya-p-sharma/]
> > <https://www.linkedin.com/in/aditya-p-sharma/>
> >
>