Hi,

On 30/01/2020 16:51, Michael Brohl wrote:
> Hi Pawan, all,
>
> I really don't see any reason to introduce tables into the codebase
> which are not used anywhere in the current functionality.
>
> I would recommend to wait until there is substantial functioanality
> which use these table. As Piere stated in some of the *Dimension
> Jiras: "The component would benefit from a ... dimension for future
> fact tables and star schema view entities".
>
> What do others think?

I'm not opposed to introduce some new dimension, if we have service to
populate them, as bi api.

Otherwise the added valuable would be small because not really easy to use.

After use a dimension with fact element are, for me, just an example how
we can use this dimension

Nicolas

>
> Thanks,
>
> Michael Brohl
>
> ecomify GmbH - www.ecomify.de
>
>
> Am 30.01.20 um 10:15 schrieb pa...@apache.org:
>> This is an automated email from the ASF dual-hosted git repository.
>>
>> pawan pushed a commit to branch trunk
>> in repository https://gitbox.apache.org/repos/asf/ofbiz-plugins.git
>>
>>
>> The following commit(s) were added to refs/heads/trunk by this push:
>>       new ccb6219  Implemented: Have a Country Dimension (OFBIZ-10954)
>> ccb6219 is described below
>>
>> commit ccb6219954a0b523b8ff854da5954af2feaab8d0
>> Author: Pawan Verma <pawan.ve...@hotwaxsystems.com>
>> AuthorDate: Thu Jan 30 14:45:25 2020 +0530
>>
>>      Implemented: Have a Country Dimension
>>      (OFBIZ-10954)
>>           Added new CountryDimension entity in olap.
>>           Thanks: Pierre Smits for your contribution.
>> ---
>>   bi/entitydef/entitygroup.xml |  1 +
>>   bi/entitydef/entitymodel.xml | 25 +++++++++++++++++++++++++
>>   2 files changed, 26 insertions(+)
>>
>> diff --git a/bi/entitydef/entitygroup.xml b/bi/entitydef/entitygroup.xml
>> index 2d34fe7..670405c 100644
>> --- a/bi/entitydef/entitygroup.xml
>> +++ b/bi/entitydef/entitygroup.xml
>> @@ -24,6 +24,7 @@ under the License.
>>       <!-- ========================================================= -->
>>       <!--
>> Dimensions                                                     -->
>>       <!-- ========================================================= -->
>> +    <entity-group group="org.apache.ofbiz.olap"
>> entity="CountryDimension"/>
>>       <entity-group group="org.apache.ofbiz.olap"
>> entity="CurrencyDimension"/>
>>       <entity-group group="org.apache.ofbiz.olap"
>> entity="DateDimension"/>
>>       <entity-group group="org.apache.ofbiz.olap"
>> entity="ProductDimension"/>
>> diff --git a/bi/entitydef/entitymodel.xml b/bi/entitydef/entitymodel.xml
>> index 57c0868..7bdfe25 100644
>> --- a/bi/entitydef/entitymodel.xml
>> +++ b/bi/entitydef/entitymodel.xml
>> @@ -29,6 +29,31 @@ under the License.
>>       <!-- ========================================================= -->
>>       <!-- Dimensions                                                -->
>>       <!-- ========================================================= -->
>> +    <entity entity-name="CountryDimension"
>> package-name="org.apache.ofbiz.bi.dimension" title="Country Dimension">
>> +        <description>Country dimension. Based on ISO 3166-1. The
>> natural key is [geoId]</description>
>> +        <field name="dimensionId" type="id">
>> +            <description>Unique identifier of the Date dimension
>> record</description>
>> +        </field>
>> +        <field name="geoId" type="id">
>> +            <description>The natural key, based on ISO 3166-1
>> alpha-3</description>
>> +        </field>
>> +        <field name="geoName" type="name">
>> +            <description>The name of the country, based on ISO
>> 3166</description>
>> +        </field>
>> +        <field name="geoCode" type="short-varchar">
>> +            <description>The ISO 3166-1 alpha-2 code of the
>> country</description>
>> +        </field>
>> +        <field name="abbreviation" type="short-varchar">
>> +            <description>Based on the ISO 3166-1 alpha-3 code of the
>> country</description>
>> +        </field>
>> +        <field name="telCountryCode" type="short-varchar">
>> +            <description>The telephone country code. Based on
>> ITU.E123 and ITU.164</description>
>> +        </field>
>> +        <field name="ccTld" type="short-varchar">
>> +            <description>The country code top-level domain for the
>> country. Base on IANA rfc1591</description>
>> +        </field>
>> +        <prim-key field="dimensionId"/>
>> +    </entity>
>>       <entity entity-name="DateDimension"
>> package-name="org.apache.ofbiz.bi.dimension" title="Date Dimension">
>>           <description>Date (days) dimension. The natural key is
>> [dateValue]</description>
>>           <field name="dimensionId" type="id">
>>
>

Attachment: pEpkey.asc
Description: application/pgp-keys

Reply via email to