loggerhead

Merge lp:~cjwatson/loggerhead/fix-marshal-version into lp:loggerhead

  1. fix-marshal-version
  2. Merge into trunk-rich
Proposed by Colin Watson
Status: Merged
Approved by: Colin Watson
Approved revision: 519
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: lp:~cjwatson/loggerhead/fix-marshal-version
Merge into: lp:loggerhead
Diff against target: 12 lines (+1/-1)
1 file modified
loggerhead/changecache.py (+1/-1)
To merge this branch: bzr merge lp:~cjwatson/loggerhead/fix-marshal-version
Reviewer Review Type Date Requested Status
Tom Wardill (community) Approve
Review via email: mp+406308@code.launchpad.net

Commit message

Fix marshal.dumps call.

Description of the change

It doesn't take keyword arguments.

To post a comment you must log in.
Revision history for this message
Tom Wardill (twom) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'loggerhead/changecache.py'
2--- loggerhead/changecache.py 2021-07-28 10:13:58 +0000
3+++ loggerhead/changecache.py 2021-07-28 14:59:08 +0000
4@@ -143,7 +143,7 @@
5 try:
6 self.cursor.execute(
7 'delete from data where key = ?', (dbapi2.Binary(key), ))
8- blob = zlib.compress(marshal.dumps(data, version=2))
9+ blob = zlib.compress(marshal.dumps(data, 2))
10 self.cursor.execute(
11 "insert into data (key, revid, data) values (?, ?, ?)",
12 list(map(dbapi2.Binary, [key, revid, blob])))

Subscribers

People subscribed via source and target branches