[ 
https://issues.apache.org/jira/browse/HDDS-1189?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16806950#comment-16806950
 ] 

Elek, Marton commented on HDDS-1189:
------------------------------------

Thanks you the patch [~swagle].

This patch introduces a lot of new dependencies. I think the LICENSE file 
should be updated.

Jooq seems to be apache 2 but HSQLDB seems to have a different licence ( and I 
didn't check the transitive dependencies of spring-jdbc).

(Ping me offline if you need help. I think the easiest way to fix this is to 
compare the distribution folder with and without this patch and check the 
licence of all the new jar files. All the non Apache should be added to the 
LICENCE. )

> Recon Aggregate DB schema and ORM
> ---------------------------------
>
>                 Key: HDDS-1189
>                 URL: https://issues.apache.org/jira/browse/HDDS-1189
>             Project: Hadoop Distributed Data Store
>          Issue Type: Sub-task
>    Affects Versions: 0.5.0
>            Reporter: Siddharth Wagle
>            Assignee: Siddharth Wagle
>            Priority: Major
>             Fix For: 0.5.0
>
>         Attachments: HDDS-1189.01.patch, HDDS-1189.02.patch, 
> HDDS-1189.03.patch, HDDS-1189.04.patch
>
>
> _Objectives_
> - Define V1 of the db schema for recon service
> - The current proposal is to use jOOQ as the ORM for SQL interaction. For two 
> main reasons: a) powerful DSL for querying, that abstracts out SQL dialects, 
> b) Allows code to schema and schema to code seamless transition, critical for 
> creating DDL through the code and unit testing across versions of the 
> application.
> - Add e2e unit tests suite for Recon entities, created based on the design doc



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to