bitbucket: simplify a bit BitBucketLister's constructor

get rid of the "smart" flush_packet_db computation.
This commit is contained in:
David Douard 2019-08-30 10:15:08 +02:00
parent d807d15f65
commit e0ce68377d

View file

@ -14,8 +14,6 @@ from swh.lister.core.indexing_lister import IndexingHttpLister
logger = logging.getLogger(__name__)
DEFAULT_BITBUCKET_PAGE = 10
class BitBucketLister(IndexingHttpLister):
PATH_TEMPLATE = '/repositories?after=%s'
@ -27,12 +25,10 @@ class BitBucketLister(IndexingHttpLister):
def __init__(self, api_baseurl, override_config=None, per_page=100):
super().__init__(
api_baseurl=api_baseurl, override_config=override_config)
if per_page != DEFAULT_BITBUCKET_PAGE:
self.PATH_TEMPLATE = '%s&pagelen=%s' % (
self.PATH_TEMPLATE, per_page)
# to stay consistent with prior behavior (20 * 10 repositories then)
self.flush_packet_db = int(
(self.flush_packet_db * DEFAULT_BITBUCKET_PAGE) / per_page)
per_page = self.config.get('per_page', per_page)
self.PATH_TEMPLATE = '%s&pagelen=%s' % (
self.PATH_TEMPLATE, per_page)
def get_model_from_repo(self, repo):
return {