[ARVADOS] updated: 635229a6fc82daa82729f602612054616874916f

git at public.curoverse.com git at public.curoverse.com
Sun Apr 5 17:39:37 EDT 2015


Summary of changes:
 sdk/python/arvados/_ranges.py | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

       via  635229a6fc82daa82729f602612054616874916f (commit)
      from  e88eac059d0c9088edf4d3f60734c4804d7ca500 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 635229a6fc82daa82729f602612054616874916f
Author: Brett Smith <brett at curoverse.com>
Date:   Sun Apr 5 17:39:33 2015 -0400

    Fix debug log formatting in PySDK.
    
    When multiple arguments are passed to a logger method, the first
    argument is expected to be a printf-style format string, with the
    remaining arguments expected to fill in the formatters.
    
    No issue #.

diff --git a/sdk/python/arvados/_ranges.py b/sdk/python/arvados/_ranges.py
index 947b35f..d4f20f0 100644
--- a/sdk/python/arvados/_ranges.py
+++ b/sdk/python/arvados/_ranges.py
@@ -96,7 +96,9 @@ def locators_and_ranges(data_locators, range_start, range_size):
         block_start = dl.range_start
         block_size = dl.range_size
         block_end = block_start + block_size
-        _logger.debug(dl.locator, "range_start", range_start, "block_start", block_start, "range_end", range_end, "block_end", block_end)
+        _logger.debug(
+            "%s range_start %s block_start %s range_end %s block_end %s",
+            dl.locator, range_start, block_start, range_end, block_end)
         if range_end <= block_start:
             # range ends before this block starts, so don't look at any more locators
             break
@@ -168,7 +170,10 @@ def replace_range(data_locators, new_range_start, new_range_size, new_locator, n
         dl = data_locators[i]
         old_segment_start = dl.range_start
         old_segment_end = old_segment_start + dl.range_size
-        _logger.debug(dl, "range_start", new_range_start, "segment_start", old_segment_start, "range_end", new_range_end, "segment_end", old_segment_end)
+        _logger.debug(
+            "%s range_start %s segment_start %s range_end %s segment_end %s",
+            dl, new_range_start, old_segment_start, new_range_end,
+            old_segment_end)
         if new_range_end <= old_segment_start:
             # range ends before this segment starts, so don't look at any more locators
             break

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list