-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Aliyun: Remove AssertHelpers #7116
Conversation
aliyun/src/test/java/org/apache/iceberg/aliyun/oss/TestOSSInputFile.java
Outdated
Show resolved
Hide resolved
aliyun/src/test/java/org/apache/iceberg/aliyun/oss/TestOSSOutputFile.java
Outdated
Show resolved
Hide resolved
aliyun/src/test/java/org/apache/iceberg/aliyun/oss/TestOSSURI.java
Outdated
Show resolved
Hide resolved
@nastra Please take a look again when you are free, thx. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, @jackye1995 could you also take a look please?
aliyun/src/test/java/org/apache/iceberg/aliyun/oss/TestOSSInputStream.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, waiting for CI
Hi, @jackye1995 The CI has passed, PTAL. |
Given this is just method deprecation, I will go ahead to merge it instead of waiting for more approvals, thanks @liuxiaocs7 and thanks @nastra for the review! |
I am working on |
Remove AssertHelpers in
iceberg-aliyun
module