search

juicedata / juicefs

JuiceFS is a distributed POSIX file system built on top of Redis and S3.
https://juicefs.com
Apache License 2.0
10.63k stars 928 forks source link

meta/sql: use binary as column type for xattr #4873

Open davies opened 4 months ago

davies commented 4 months ago

close #4803

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 54.52%. Comparing base (fc2d91d) to head (878d184). Report is 15 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #4873 +/- ## ========================================== + Coverage 44.86% 54.52% +9.65% ========================================== Files 158 160 +2 Lines 42872 43351 +479 ========================================== + Hits 19233 23635 +4402 + Misses 21228 17036 -4192 - Partials 2411 2680 +269 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

SandyXSD commented 4 months ago

We have to update the table schema manually if we want this change, see: https://juicefs.com/docs/community/release_notes/#juicefs-v10

SandyXSD commented 4 months ago 
# sqlite as the meta engine
# old version
root@dev:/mnt/jfs# xattr -w testkey testval f
root@dev:/mnt/jfs# xattr -l f
testkey: testval
# with this PR
root@dev:/mnt/jfs# xattr -l f
No such xattr: testkey