Skip to content

HIVE-28909: Modify McolumnDescriptor definition in package.jdo to sup… - #5804

Merged
ayushtkn merged 8 commits into
apache:masterfrom
saihemanth-cloudera:hive-28909
May 23, 2025
Merged

HIVE-28909: Modify McolumnDescriptor definition in package.jdo to sup…#5804
ayushtkn merged 8 commits into
apache:masterfrom
saihemanth-cloudera:hive-28909

Conversation

@saihemanth-cloudera

Copy link
Copy Markdown
Contributor

…port JDK17 + JDO6.0

For Hive to support JDK17 + JDO6.0, select/update/delete queries of the table are failing to fetch column information of the table. The DataNucleus6.0 is unable to map the Mcolumn Descriptor definition to the underlying table schema due to the presence of CLOB type field for the join condition in the package.jdo file. This patch addresses the issue by avoiding the CLOB column in the join condition.

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

…port JDK17 + JDO6.0

For Hive to support JDK17 + JDO6.0, select/update/delete queries of the table are failing
to fetch column information of the table. The DataNucleus6.0 is unable to map the Mcolumn
Descriptor definition to the underlying table schema due to the presence of CLOB type field
for the join condition in the package.jdo file. This patch addresses the issue by avoiding
the CLOB column in the join condition.

@dengzhhu653 dengzhhu653 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 pending tests

@deniskuzZ deniskuzZ left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

Comment thread standalone-metastore/metastore-server/src/main/resources/package.jdo Outdated
@sonarqubecloud

Copy link
Copy Markdown

@ayushtkn
ayushtkn merged commit fbe090f into apache:master May 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants