aboutsummaryrefslogtreecommitdiff
path: root/weed/filer
diff options
context:
space:
mode:
authorChris Lu <chris.lu@gmail.com>2020-09-24 11:11:42 -0700
committerChris Lu <chris.lu@gmail.com>2020-09-24 11:11:42 -0700
commit1012df7bb55341fb7d835269cf9bb7edc6507d25 (patch)
tree4ec73d9833a48dd1210e99e19bc935454d9815ee /weed/filer
parent4856bce0ee929088c64e311b4ac554e872fba12e (diff)
downloadseaweedfs-1012df7bb55341fb7d835269cf9bb7edc6507d25.tar.xz
seaweedfs-1012df7bb55341fb7d835269cf9bb7edc6507d25.zip
switch hardlink id from int64 to bytes
Diffstat (limited to 'weed/filer')
-rw-r--r--weed/filer/entry.go2
-rw-r--r--weed/filer/entry_codec.go6
-rw-r--r--weed/filer/filer_delete_entry.go9
-rw-r--r--weed/filer/filerstore.go2
-rw-r--r--weed/filer/filerstore_hardlink.go15
5 files changed, 15 insertions, 19 deletions
diff --git a/weed/filer/entry.go b/weed/filer/entry.go
index 45ede0e8e..421e51432 100644
--- a/weed/filer/entry.go
+++ b/weed/filer/entry.go
@@ -64,7 +64,7 @@ func (entry *Entry) ToProtoEntry() *filer_pb.Entry {
Attributes: EntryAttributeToPb(entry),
Chunks: entry.Chunks,
Extended: entry.Extended,
- HardLinkId: int64(entry.HardLinkId),
+ HardLinkId: entry.HardLinkId,
HardLinkCounter: entry.HardLinkCounter,
}
}
diff --git a/weed/filer/entry_codec.go b/weed/filer/entry_codec.go
index 21531ad7a..884fb2670 100644
--- a/weed/filer/entry_codec.go
+++ b/weed/filer/entry_codec.go
@@ -16,7 +16,7 @@ func (entry *Entry) EncodeAttributesAndChunks() ([]byte, error) {
Attributes: EntryAttributeToPb(entry),
Chunks: entry.Chunks,
Extended: entry.Extended,
- HardLinkId: int64(entry.HardLinkId),
+ HardLinkId: entry.HardLinkId,
HardLinkCounter: entry.HardLinkCounter,
}
return proto.Marshal(message)
@@ -36,7 +36,7 @@ func (entry *Entry) DecodeAttributesAndChunks(blob []byte) error {
entry.Chunks = message.Chunks
- entry.HardLinkId = HardLinkId(message.HardLinkId)
+ entry.HardLinkId = message.HardLinkId
entry.HardLinkCounter = message.HardLinkCounter
return nil
@@ -116,7 +116,7 @@ func EqualEntry(a, b *Entry) bool {
}
}
- if a.HardLinkId != b.HardLinkId {
+ if !bytes.Equal(a.HardLinkId, b.HardLinkId) {
return false
}
if a.HardLinkCounter != b.HardLinkCounter {
diff --git a/weed/filer/filer_delete_entry.go b/weed/filer/filer_delete_entry.go
index 693b93f8b..6c9ff56d3 100644
--- a/weed/filer/filer_delete_entry.go
+++ b/weed/filer/filer_delete_entry.go
@@ -10,12 +10,7 @@ import (
"github.com/chrislusf/seaweedfs/weed/util"
)
-type HardLinkId int64
-func (hardLinkId HardLinkId) Key() []byte{
- bytes := make([]byte, 8)
- util.Uint64toBytes(bytes, uint64(hardLinkId))
- return bytes
-}
+type HardLinkId []byte
func (f *Filer) DeleteEntryMetaAndData(ctx context.Context, p util.FullPath, isRecursive, ignoreRecursiveError, shouldDeleteChunks, isFromOtherCluster bool, signatures []int32) (err error) {
if p == "/" {
@@ -95,7 +90,7 @@ func (f *Filer) doBatchDeleteFolderMetaAndData(ctx context.Context, entry *Entry
hardlinkIds = append(hardlinkIds, dirHardLinkIds...)
} else {
f.NotifyUpdateEvent(ctx, sub, nil, shouldDeleteChunks, isFromOtherCluster, nil)
- if sub.HardLinkId != 0 {
+ if len(sub.HardLinkId) != 0 {
// hard link chunk data are deleted separately
hardlinkIds = append(hardlinkIds, sub.HardLinkId)
} else {
diff --git a/weed/filer/filerstore.go b/weed/filer/filerstore.go
index 888168581..11e30878d 100644
--- a/weed/filer/filerstore.go
+++ b/weed/filer/filerstore.go
@@ -135,7 +135,7 @@ func (fsw *FilerStoreWrapper) DeleteEntry(ctx context.Context, fp util.FullPath)
if findErr == filer_pb.ErrNotFound {
return nil
}
- if existingEntry.HardLinkId != 0 {
+ if len(existingEntry.HardLinkId) != 0 {
// remove hard link
if err = fsw.DeleteHardLink(ctx, existingEntry.HardLinkId); err != nil {
return err
diff --git a/weed/filer/filerstore_hardlink.go b/weed/filer/filerstore_hardlink.go
index ec768b9cb..9cb32f27f 100644
--- a/weed/filer/filerstore_hardlink.go
+++ b/weed/filer/filerstore_hardlink.go
@@ -1,6 +1,7 @@
package filer
import (
+ "bytes"
"context"
"fmt"
"github.com/chrislusf/seaweedfs/weed/glog"
@@ -8,7 +9,7 @@ import (
)
func (fsw *FilerStoreWrapper) handleUpdateToHardLinks(ctx context.Context, entry *Entry) error {
- if entry.HardLinkId == 0 {
+ if len(entry.HardLinkId) == 0 {
return nil
}
// handle hard links
@@ -23,7 +24,7 @@ func (fsw *FilerStoreWrapper) handleUpdateToHardLinks(ctx context.Context, entry
}
// remove old hard link
- if err == nil && existingEntry.HardLinkId != entry.HardLinkId {
+ if err == nil && bytes.Compare(existingEntry.HardLinkId, entry.HardLinkId) != 0 {
if err = fsw.DeleteHardLink(ctx, existingEntry.HardLinkId); err != nil {
return err
}
@@ -32,10 +33,10 @@ func (fsw *FilerStoreWrapper) handleUpdateToHardLinks(ctx context.Context, entry
}
func (fsw *FilerStoreWrapper) setHardLink(ctx context.Context, entry *Entry) error {
- if entry.HardLinkId == 0 {
+ if len(entry.HardLinkId) == 0 {
return nil
}
- key := entry.HardLinkId.Key()
+ key := entry.HardLinkId
newBlob, encodeErr := entry.EncodeAttributesAndChunks()
if encodeErr != nil {
@@ -46,10 +47,10 @@ func (fsw *FilerStoreWrapper) setHardLink(ctx context.Context, entry *Entry) err
}
func (fsw *FilerStoreWrapper) maybeReadHardLink(ctx context.Context, entry *Entry) error {
- if entry.HardLinkId == 0 {
+ if len(entry.HardLinkId) == 0 {
return nil
}
- key := entry.HardLinkId.Key()
+ key := entry.HardLinkId
value, err := fsw.KvGet(ctx, key)
if err != nil {
@@ -66,7 +67,7 @@ func (fsw *FilerStoreWrapper) maybeReadHardLink(ctx context.Context, entry *Entr
}
func (fsw *FilerStoreWrapper) DeleteHardLink(ctx context.Context, hardLinkId HardLinkId) error {
- key := hardLinkId.Key()
+ key := hardLinkId
value, err := fsw.KvGet(ctx, key)
if err == ErrKvNotFound {
return nil