diff options
| author | Peter Portante <peter.portante@redhat.com> | 2012-12-09 23:56:28 -0500 | 
|---|---|---|
| committer | Vijay Bellur <vbellur@redhat.com> | 2012-12-17 06:11:18 -0800 | 
| commit | 2fd8af2b750c43c657df9d8f9ba6fd5c1ba1f437 (patch) | |
| tree | 7c2c513e7c75663c20a61ba36888d67594ee17ba /libglusterfs | |
| parent | 47e21afaaf192b03db69d8f204b3e33e5f9596cf (diff) | |
object-storage: a set of cleanups to DiskFile
Here are a set of cleanups for things noticed while working on the
unit test coverage:
  * Remove unused constants, adding a constant for default disk chunk
    size
  * Document missing logger parameter from constructor
  * Add iter_hook paramater to constructor in anticipation of 1.7.4
    support
  * Add back meta_file field even though it is not used in our version
    of the constructor for paranoid compatibility
  * Remove is_valid field as it is not referenced anywhere, and it is
    not a field of the super class
  * Rename fields only used internally by DiskFile with leading
    underscores
  * Convert to using os.path.join() instead of hard coded '/'
    references
  * Use data_file field where possible
  * Assert that put_metadata() will only work when the file exists
  * Remove update_object() method since it is identical to
    put_metadata() and a direct write will suffice
  * Create the directory object only if it does not exist when a
    marker directory is requested
Change-Id: If207fed4c0f423e6bd3232e4507f87de877057c4
BUG: 876660
Signed-off-by: Peter Portante <peter.portante@redhat.com>
Reviewed-on: http://review.gluster.org/4287
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Mohammed Junaid <junaid@redhat.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'libglusterfs')
0 files changed, 0 insertions, 0 deletions
