Skip to content

Commit

Permalink
Set batch size to 800 to resolve sqlite issues
Browse files Browse the repository at this point in the history
Prevents Pulp from running into sqlite's evaluation tree size limit.

closes #3499
https://pulp.plan.io/issues/3499
  • Loading branch information
dralley committed Mar 20, 2018
1 parent 855d202 commit 5ade473
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion plugin/pulpcore/plugin/changeset/iterator.py
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,8 @@ def _collated_content(self):
dict: A dictionary of {model_class: [content,]}
Each content is: PendingContent.
"""
for batch in BatchIterator(self.content, 1024):
# should be < 1000 as higher values may hit the sqlite expression tree size limit
for batch in BatchIterator(self.content, 800):
collated = {}
for content in batch:
_list = collated.setdefault(type(content.model), list())
Expand Down

0 comments on commit 5ade473

Please sign in to comment.