You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by GitBox <gi...@apache.org> on 2019/12/12 05:04:23 UTC

[GitHub] [cloudstack] cl-k-takahashi opened a new issue #3762: [DB] cloud.host_pod_ref.description is not 1NF.

cl-k-takahashi opened a new issue #3762: [DB] cloud.host_pod_ref.description is not 1NF.
URL: https://github.com/apache/cloudstack/issues/3762
 
 
   <!--
   Verify first that your issue/request is not already reported on GitHub.
   Also test if the latest release and master branch are affected too.
   Always add information AFTER of these HTML comments, but no need to delete the comments.
   -->
   
   ##### ISSUE TYPE
   <!-- Pick one below and delete the rest -->
    * Improvement Request
   
   ##### COMPONENT NAME
   <!--
   Categorize the issue, e.g. API, VR, VPN, UI, etc.
   -->
   ~~~
   DB schema
   ~~~
   
   ##### CLOUDSTACK VERSION
   <!--
   New line separated list of affected versions, commit ID for issues on master branch.
   -->
   
   ~~~
   master
   ~~~
   
   ##### SUMMARY
   <!-- Explain the problem/feature briefly -->
   cloud.host_pod_ref.description may contain multiple tuples. This is not 1NF.
   ```
                       /*
                        * POD Description is refactored to:
                        * <START_IP>-<END_IP>-<FOR_SYSTEM_VMS>-<VLAN>,<START_IP>-<END_IP>-<FOR_SYSTEM_VMS>-<VLAN>,...
                       */
   ```
   I propose to use separate ip range table like this:
   ```
   mysql> desc pod_ip_range_map;
   +----------------+---------------------+------+-----+---------+----------------+
   | Field          | Type                | Null | Key | Default | Extra          |
   +----------------+---------------------+------+-----+---------+----------------+
   | id             | bigint(20) unsigned | NO   | PRI | NULL    | auto_increment |
   | pod_id         | bigint(20) unsigned | YES  | MUL | NULL    |                |
   | start_ip       | varchar(15)         | YES  |     | NULL    |                |
   | end_ip         | varchar(15)         | YES  |     | NULL    |                |
   | for_system_vms | tinyint(1)          | YES  |     | NULL    |                |
   | vlan_id        | varchar(255)        | YES  |     | NULL    |                |
   +----------------+---------------------+------+-----+---------+----------------+
   6 rows in set (0.00 sec)
   ```
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services