Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(30)

Issue 6494142: whoosh create/destroy storage api

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 years, 5 months ago by Thomas.J.Waldmann
Modified:
2 years, 4 months ago
Reviewers:
matt2, thomas.j.waldmann, thomas.j.waldmann
Visibility:
Public.

Patch Set 1 #

Total comments: 21
Unified diffs Side-by-side diffs Delta from patch set Stats (+297 lines, -11 lines) Patch
M src/whoosh/filedb/filestore.py View 13 chunks +297 lines, -11 lines 21 comments Download

Messages

Total messages: 4
ThomasJWaldmann
hi matt, i think it is easier to discuss such stuff on codereview, please have ...
2 years, 5 months ago (2012-09-15 14:00:29 UTC) #1
matt2
I'm not sure if this is using Reitveld the right way, but I've replied here ...
2 years, 5 months ago (2012-09-18 19:20:27 UTC) #2
ThomasJWaldmann
some clarifications / replies. if you refine this, can you try to update this codereview? ...
2 years, 5 months ago (2012-09-19 09:06:01 UTC) #3
matt2
2 years, 4 months ago (2012-10-11 20:47:01 UTC) #4
> I was referring to close() (not destroy()) and usage like this:

Sorry for the misunderstanding. This is checked in now.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1437:571d95b2634e