[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