Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remaining patches #3

Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open

Remaining patches #3

wants to merge 22 commits into from

Conversation

frett
Copy link
Contributor

@frett frett commented Jan 31, 2013

These are all of my remaining patches that I hadn't submitted yet.

  1. creation of a Node object similar to the MogileFS::FID object.
  2. addition of a filepaths_remove_directory command.
  3. add path_to_key support to cmd_updateclass.

I would have sent them individually, but 2 is dependent on 1 and 3 would have included all the history from 1 in the pull request

@frett
created a Node object that will represent a FilePaths node
9768ad4
@frett
updated _traverse_path to return a Node object
ca2eed4
@frett
updated cmd_delete hook to use the node object
36387d2
@frett
removed unused delete_file_mapping function
d420188
@frett
get rid of code duplication
7fabdd8
@frett
reworked the file_stored hook to use less db queries and be safer abo…
f599efd 
…ut deleting the old FID if something fails
@frett
update the filepaths_rename command to use the get_node_by_parent met…
fa55a7e 
…hod and not vivify the path if it can't find the source path
@frett
make vivify_path return the node at the end of the path
c86867d
@frett
rewrite the list_directory command to use less db queries
3ecfe79
@frett
updated _path_to_key to utilize the node object
8122cc4
@frett
remove a few unused MogileFS::Store methods
03123a7
@frett
update load_path to return a node object
6cd432a
@frett
remove a couple redundant tests
2a84e2f
@frett
unregister all hooks
99346cc
@frett
make sure path traversal only processes directory nodes
13d37e1
@frett
do a bit more validation of when we are working on a file vs a directory
20e0ab0
@frett
added a remove_directory command that can be used to delete empty dir…
32dc338 
…ectories
@frett
only retrieve the fid if the node exists
111845a
@frett
update the key for the updateclass cmd
9006fd0
@frett
Merge branches 'rmDir' and 'updateClass' into master
77de2e0
@frett
utilize the get_bulk_metadata method
d32daee
@frett
Merge branch 'bulkMeta' into remainingPatches
4bf00a4
@frett
Copy link
Contributor Author

frett commented May 18, 2013

commit d32daee utilizes a new MetaData method added with this pull request: hachi/MogileFS-Plugin-MetaData#1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@frett