You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hive.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2020/07/01 16:32:00 UTC
[jira] [Work logged] (HIVE-23363) Upgrade DataNucleus dependency to
5.2
[ https://issues.apache.org/jira/browse/HIVE-23363?focusedWorklogId=453508&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-453508 ]
ASF GitHub Bot logged work on HIVE-23363:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 01/Jul/20 16:31
Start Date: 01/Jul/20 16:31
Worklog Time Spent: 10m
Work Description: ashutoshc commented on a change in pull request #1118:
URL: https://github.com/apache/hive/pull/1118#discussion_r448480980
##########
File path: standalone-metastore/metastore-server/src/main/resources/package.jdo
##########
@@ -345,20 +345,20 @@
<datastore-identity>
<column name="CD_ID"/>
</datastore-identity>
- <field name="cols" table="COLUMNS_V2" >
Review comment:
Can you describe the need for this change?
##########
File path: standalone-metastore/metastore-server/src/main/resources/package.jdo
##########
@@ -345,20 +345,20 @@
<datastore-identity>
<column name="CD_ID"/>
</datastore-identity>
- <field name="cols" table="COLUMNS_V2" >
+ <field name="cols" persistence-modifier="persistent" table="COLUMNS_V2">
<collection element-type="MFieldSchema"/>
- <foreign-key name="COLUMNS_V2_FK" delete-action="cascade"/>
<join>
- <primary-key name="COLUMNS_PK">
Review comment:
Any reason to change the name here?
##########
File path: standalone-metastore/metastore-server/src/main/resources/package.jdo
##########
@@ -345,20 +345,20 @@
<datastore-identity>
<column name="CD_ID"/>
</datastore-identity>
- <field name="cols" table="COLUMNS_V2" >
+ <field name="cols" persistence-modifier="persistent" table="COLUMNS_V2">
<collection element-type="MFieldSchema"/>
- <foreign-key name="COLUMNS_V2_FK" delete-action="cascade"/>
Review comment:
We do want cascade-delete here. Any reason to remove it?
##########
File path: standalone-metastore/metastore-server/src/main/resources/package.jdo
##########
@@ -345,20 +345,20 @@
<datastore-identity>
<column name="CD_ID"/>
</datastore-identity>
- <field name="cols" table="COLUMNS_V2" >
+ <field name="cols" persistence-modifier="persistent" table="COLUMNS_V2">
<collection element-type="MFieldSchema"/>
- <foreign-key name="COLUMNS_V2_FK" delete-action="cascade"/>
<join>
- <primary-key name="COLUMNS_PK">
+ <primary-key name="SQL110922153006740">
+ <column name="CD_ID"/>
Review comment:
This probably is fine to do. Though, was it necessary?
----------------------------------------------------------------
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
Issue Time Tracking
-------------------
Worklog Id: (was: 453508)
Time Spent: 40m (was: 0.5h)
> Upgrade DataNucleus dependency to 5.2
> -------------------------------------
>
> Key: HIVE-23363
> URL: https://issues.apache.org/jira/browse/HIVE-23363
> Project: Hive
> Issue Type: Improvement
> Affects Versions: 4.0.0
> Reporter: Zoltan Chovan
> Assignee: Zoltan Chovan
> Priority: Critical
> Labels: pull-request-available
> Attachments: HIVE-23363.2.patch, HIVE-23363.patch
>
> Time Spent: 40m
> Remaining Estimate: 0h
>
> Upgrade Datanucleus from 4.2 to 5.2 as based on it's docs 4.2 has been retired:
> [http://www.datanucleus.org/documentation/products.html]
--
This message was sent by Atlassian Jira
(v8.3.4#803005)