Index files aren't sync'd when user calls plfs_sync

[brettkettering]

John Bent noted that it was his understanding that index files aren't getting sync'd when user calls plfs_sync. Everything needs to be synced to storage on plfs_sync, not just data.

[johnbent]

Michael Kukat first noticed this and is interested in this.

On Apr 9, 2014, at 10:53 AM, Brett Kettering [email protected] wrote:

John Bent noted that it was his understanding that index files aren't getting sync'd when user calls plfs_sync. Everything needs to be synced to storage on plfs_sync, not just data.

—
Reply to this email directly or view it on GitHub.

[johnbent]

Hi,

i had a quick look into the sources and it looks like the index files
should also be flushed&synced, so i'm not 100% sure if there really is
an issue in 2.5. At least the same test (crashing one node during
writing to a shared file with known sync intervals, so the other node
can take over a at a safe position before where the crashed one left)
worked fine with 2.3.
Didn't perform enough test runs with 2.3 for good statistics, but for
2.5, i had 2 or 3 runs and the index files of the crashed node always
were 0 bytes afterwards, the file was not readable.

...Michael

Am 09.04.2014 17:15, schrieb John Bent:

Michael Kukat first noticed this and is interested in this.

On Apr 9, 2014, at 10:53 AM, Brett Kettering <[email protected]
mailto:[email protected]> wrote:

John Bent noted that it was his understanding that index files aren't
getting sync'd when user calls plfs_sync. Everything needs to be
synced to storage on plfs_sync, not just data.

—
Reply to this email directly or view it on GitHub
#349.

signature german Mit freundlichen Grüßen

Michael Kukat
Senior Software Developer

GRAU DATA AG
Marie-Curie-Str. 19
73529 Schwäbisch Gmünd

E-Mail: [email protected] mailto:[email protected]
Tel.: +49 7171 187-0
Fax: +49 7171 187-250

GRAU DATA - YOU DATA, YOUR CONTROL

Amtsgericht ULM: HRB 701807; USt-ID-Nr.: DE210200743; Vorstand: Herbert
Grau, CTO Ulrich Lechner; Aufsichtsratsvorsitzender: Friedhelm Steinhilber