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

[Bug][Core]: The PartitionPropertiesUpdate can't remove partition properties key #1107

Closed
1 task done
hzluting opened this issue Feb 14, 2023 · 0 comments · Fixed by #1108
Closed
1 task done

[Bug][Core]: The PartitionPropertiesUpdate can't remove partition properties key #1107

hzluting opened this issue Feb 14, 2023 · 0 comments · Fixed by #1108
Labels
priority:blocker security, data-loss, correctness, etc. type:bug Something isn't working
Milestone

Comments

@hzluting
Copy link
Contributor

What happened?

When I use the PartitionPropertiesUpdate#remove method, it is not effective.

Affects Versions

0.3.0

What engines are you seeing the problem on?

No response

How to reproduce

No response

Relevant log output

No response

Anything else

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@hzluting hzluting added the type:bug Something isn't working label Feb 14, 2023
hzluting pushed a commit to hzluting/arctic that referenced this issue Feb 14, 2023
add remove key to removeProperties
hzluting pushed a commit to hzluting/arctic that referenced this issue Feb 14, 2023
add remove key to removeProperties
@hzluting hzluting added this to the Release 0.4.1 milestone Feb 14, 2023
@hzluting hzluting added the priority:blocker security, data-loss, correctness, etc. label Feb 14, 2023
zhoujinsong pushed a commit that referenced this issue Feb 20, 2023
…move key (#1108)

* fix #1107
add remove key to removeProperties

* fix #1107
add remove key to removeProperties

* fix import

* fix code review

---------

Co-authored-by: luting <[email protected]>
zhoujinsong pushed a commit that referenced this issue May 31, 2023
…move key (#1108)

* fix #1107
add remove key to removeProperties

* fix #1107
add remove key to removeProperties

* fix import

* fix code review

---------

Co-authored-by: luting <[email protected]>
ShawHee pushed a commit to ShawHee/arctic that referenced this issue Dec 29, 2023
…move key (apache#1108)

* fix apache#1107
add remove key to removeProperties

* fix apache#1107
add remove key to removeProperties

* fix import

* fix code review

---------

Co-authored-by: luting <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:blocker security, data-loss, correctness, etc. type:bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant