Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Store papers in LazyDb as dict instead of [], pkey of str(pid) #8

Open
mekarpeles opened this issue Feb 8, 2013 · 1 comment
Open

Store papers in LazyDb as dict instead of [], pkey of str(pid) #8

mekarpeles opened this issue Feb 8, 2013 · 1 comment
Assignees
@mekarpeles
Labels
invalid

Comments

@mekarpeles
Copy link
Owner

No description provided.

@ghost ghost assigned mekarpeles Feb 8, 2013
@mekarpeles
Copy link
Owner Author

After further thought, this could be a poor architectural decision as it prevents us from paginating in a deterministic way. It would require that we sort the dict; sorted(papers, lambda x: ...).

The advantage would be, if the key (paper id or comment id) changes, only a singly entry and its direct relationships are affected (namely a paper and its comments), This risk can be mitigated by having a policy where no paper is deleted, only flagged as disabled (enabled=False).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mekarpeles mekarpeles

Labels
invalid
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mekarpeles