Conversation
zhoujinsong
left a comment
zhoujinsong
left a comment
There was a problem hiding this comment.
@czy006 I left a comment and the ci seems fail.
| <artifactId>lance-namespace-apache-client</artifactId> | ||
| </exclusion> | ||
| </exclusions> | ||
| <version>8.0.0</version> |
There was a problem hiding this comment.
How about adding a version property?
|
This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@amoro.apache.org list. Thank you for your contributions. |
|
This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@amoro.apache.org list. Thank you for your contributions. |
2 participants
Why are the changes needed?
Upgrade Lance dependency to 8.0.0
Brief change log
How was this patch tested?
Add some test cases that check the changes thoroughly including negative and positive cases if possible
Add screenshots for manual tests if appropriate
Run test locally before making a pull request
Documentation