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?

Thanks,

Michael Brohl

ecomify GmbH - www.ecomify.de


Am 30.01.20 um 10:15 schrieb [email protected]:
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 <[email protected]>
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: smime.p7s
Description: S/MIME Cryptographic Signature

Reply via email to