Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DBT-703: Added tests for Iceberg v2 format #174

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

niteshy
Copy link
Collaborator

@niteshy niteshy commented Jun 8, 2023

Describe your changes

To test the iceberg v2 table are supported, added tests for it.

Internal Jira ticket number or external issue link

https://jira.cloudera.com/browse/DBT-703

Testing procedure/screenshots(if appropriate):

https://gist.github.com/niteshy/62cecfbf4af78189f72980e49b7b2d9f

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have formatted my added/modified code to follow pep-8 standards
  • I have checked suggestions from python linter to make sure code is of good quality.

check_result_nodes_by_name,
relation_from_name,
check_relation_types,
)


def is_iceberg_table(project, tableName):
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we add a function to make sure the created table is of format v2.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The verification of table format-version is blocked by https://issues.apache.org/jira/browse/IMPALA-12209, as impala currently doesn't provide the format-version in the output.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants