Skip to content

Conversation

@deniskuzZ
Copy link
Member

@deniskuzZ deniskuzZ commented Jun 28, 2023

What changes were proposed in this pull request?

Disallow the creation of a Temporary Hive Iceberg tables

Why are the changes needed?

Atlas by default skips ingesting temporary tables created in Hive. But when searched for the newly created temporary Hive tables, it displays them in the search results.

Does this PR introduce any user-facing change?

No

Is the change a dependency upgrade?

How was this patch tested?

Unit test

Copy link
Member

@simhadri-g simhadri-g left a comment

Choose a reason for hiding this comment

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

LGTM +1 (non-binding)
Please resolve merge confilts.

Copy link
Member

@ayushtkn ayushtkn left a comment

Choose a reason for hiding this comment

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

LGTM

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 9 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@deniskuzZ deniskuzZ merged commit e489cf3 into apache:master Jul 11, 2023
@deniskuzZ deniskuzZ deleted the HIVE-27472 branch July 11, 2023 05:31
yeahyung pushed a commit to yeahyung/hive that referenced this pull request Jul 20, 2023
…menko, reviewed by Attila Turoczy, Ayush Saxena, Simhadri Govindappa)

Closes apache#4456
tarak271 pushed a commit to tarak271/hive-1 that referenced this pull request Dec 19, 2023
…menko, reviewed by Attila Turoczy, Ayush Saxena, Simhadri Govindappa)

Closes apache#4456
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.

5 participants