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.

Sign up for GitHub

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

Jump to bottom

Refactor testTimestampCoercion* in Delta Lake connector #22042

Open
ebyhr opened this issue May 21, 2024 · 1 comment
Open

Refactor testTimestampCoercion* in Delta Lake connector #22042

ebyhr opened this issue May 21, 2024 · 1 comment
Labels
delta-lake Delta Lake connector good first issue Good for newcomers

Comments

@ebyhr
Copy link
Member

ebyhr commented May 21, 2024 •
edited
Loading

We should introduce TypeCoercionTestSetup record which is used in BaseIcebergConnectorTest.
Also, nested types (e.g. array(array(...))) should be tested after the refactoring.
@ebyhr ebyhr added good first issue Good for newcomers delta-lake Delta Lake connector labels May 21, 2024
@ebyhr ebyhr mentioned this issue May 21, 2024
Merged
@adrianna-yixin
Copy link

Hi @ebyhr, I'd like to work on this issue. 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
delta-lake Delta Lake connector good first issue Good for newcomers
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ebyhr @adrianna-yixin