Opened 7 years ago

Last modified 6 months ago

#8047 assigned enhancement

BFS allows creation of indices on no-index partitions

Reported by: anevilyak Owned by: nobody
Priority: low Milestone: Unscheduled
Component: File Systems/BFS Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

If a partition has been created using the noindex parameter, one would normally expect such a partition to also disallow creation of any subsequent indices. This currently isn't the case, and one can thus wind up with a partition that lacks the usual built-in indices such as name, last_modified, etc. but has e.g. be:deskbar_item_status and mail indices. This should probably be forbidden by the FS on such a volume, perhaps also providing a tool which allows one to later switch the partition between modes if one changes one's mind. By implication, switching from non to indexed would then create all the built-in indices that one cannot create manually.

Change History (6)

comment:1 Changed 2 years ago by axeld

Not sure this qualifies as a bug; at least it's an intentional feature :-)

You should be able to recreate the indices whenever you want; an unused index doesn't really cost anything (but an empty directory). However, I agree that it would be nice to have a flag that actually prevents index creation.

comment:2 Changed 2 years ago by anevilyak

In that case, one would also want some tool or a flag on mkindex to recreate the special indices though.

comment:3 Changed 2 years ago by axeld

Type: bugenhancement

Yes, that would be nice to have. I'm not sure if the reindex tool can be of any help with those -- probably not. I'm changing this to 'enhancement', as the current behavior is intentional, even if debatable.

comment:4 Changed 2 years ago by anevilyak

Looking at reindex's current code, it would AFAICT not. All it does is read, remove, and write back all attributes on each file, which if memory serves, wouldn't alter the last modification time, so it would also need to be adjusted to be able to accommodate the mtime if/when we wind up supporting this.

comment:5 Changed 2 years ago by axeld

Owner: changed from axeld to nobody
Status: newassigned

comment:6 Changed 6 months ago by waddlesplash

Milestone: R1Unscheduled
Note: See TracTickets for help on using tickets.