bugGluster - Bugs: bug #17295, xlator to be renamed

 
 

bug #17295: xlator to be renamed

Submitter:  Anand Avati <avati>
Submitted:  Thu 03 Aug 2006 06:30:19 PM UTC
   
 
Category:  GlusterFS Severity:  2 - Minor
Priority:  5 - Normal Item Group:  Bad naming convention
Status:  Fixed Privacy:  Public
Assigned to:  avati Originator Name:  *
Open/Closed:  Closed Release: 
Operating System:  GNU/Linux Reproducibility:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Fri 11 Aug 2006 07:51:29 PM UTC, comment #2: 

all xlators are now in lower case.
xlators have been renamed as category/name. there is a top level category directory and sub directory for actual xlator.

summary of categories and renaming done -

1. Transport category (xport)
   Brick -> xport/socket

2. Optimizers/performance booster category (steroid)
   getattr -> steroid/stat-prefetch

3. Storage/File system category (storage)
   Posix -> storage/posix

4. Schedulers category (sched)
   Cement -> sched/unify


filter xlator is under discussion for its design still. One proposal -

--porposal--

5. Filtering category (filter)
   filter/read-only
   filter/hide
   filter/block
   filter/file-size-limit

each of those will be seperate translators under the filter category. more discussion to be done about chaining of multiple xlators of filter or group them into one (and effect of both approaches) to be done.
--end-proposal--


since all translators so far are following the discipline of declaring all functions  static there is no namespace issues (yet).

all inner variable and function names are to be renamed by
translator authors to follow the new naming convention

Anand Avati <avati>
Group Member
Thu 03 Aug 2006 08:53:19 PM UTC, comment #1: 

should update -

  • Makefile.am (all of them)
  • configure.ac
  • rename directories
  • rename files


Anand Avati <avati>
Group Member
Thu 03 Aug 2006 06:30:19 PM UTC, original submission:  

xlator names  should be of the type 'Category-Name'

Brick => 'Transport-Socket'
Posix => 'FS-Posix'
Cement => 'Sched-Union'

Anand Avati <avati>
Group Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

Only logged-in users can vote.

 

Follow 4 latest changes.

Date Changed by Updated Field Previous Value => Replaced by
2006-08-11 avati StatusNone Fixed
    Open/ClosedOpen Closed
2006-08-04 avati Assigned togowda avati
2006-08-03 avati Assigned toNone gowda

Back to the top

Powered by Savane 3.13-caa5.
Corresponding source code