Project

General

Profile

Actions

Feature #12106

open

CInodes encoded unversioned in dirfrags

Added by John Spray almost 9 years ago. Updated almost 8 years ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
Code Hygiene
Target version:
-
% Done:

0%

Source:
other
Tags:
Backport:
Reviewed:
Affected Versions:
Component(FS):
MDS
Labels (FS):
Pull request ID:

Description

Where we encode CInodes in the omap values of a dirfrag, we do it without any ENCODE_START decorators (InodeStoreBase::encode_bare).

This results in wonky decoding logic where we're reading til the end of the buffer instead of reading fields expected for a particular version. Just about works for fixed size fields, but is undesirable. We should take the small overhead of encoding version headers in each inode.

Actions #1

Updated by John Spray almost 9 years ago

  • Tracker changed from Bug to Feature
Actions #2

Updated by Greg Farnum almost 8 years ago

  • Category changed from 47 to Code Hygiene
  • Component(FS) MDS added
Actions

Also available in: Atom PDF