Add revision log field in the revision type

Storage is returning a list of dicts for the revision log Add a from_dict converter to create the model objects from these dicts This is not efficient and adds an extra iteration. Has to be fixed in the storage in the future.

Related to #4366 (closed)

---

Migrated from D8310 (view on Phabricator)

Build is green

Patch application report for D8310 (id=29993)

Rebasing onto 6fb1913f...

Current branch diff-target is up to date.

Changes applied before test

commit e21947ad862f0a75edbff1c87607d85e01a61fe9
Author: Jayesh Velayudhan <jayesh@softwareheritage.org>
Date:   Wed Aug 24 17:04:56 2022 +0200

    Add revision log field in the revision type
    
    Storage is returning a list of dicts for the revision log
    Add a from_dict converter to create the model objects from these dicts
    This is not efficient and adds an extra iteration. Has to be
    fixed in the storage in the future.
    
    Related to #4366

See https://jenkins.softwareheritage.org/job/DGQL/job/tests-on-diff/122/ for more details.

Remove unnecessary comment

Build is green

Patch application report for D8310 (id=29994)

Rebasing onto 6fb1913f...

Current branch diff-target is up to date.

Changes applied before test

commit 12003a050365d3570b4e3d8ba9b0bf34810b0a1d
Author: Jayesh Velayudhan <jayesh@softwareheritage.org>
Date:   Wed Aug 24 17:04:56 2022 +0200

    Add revision log field in the revision type
    
    Storage is returning a list of dicts for the revision log
    Add a from_dict converter to create the model objects from these dicts
    This is not efficient and adds an extra iteration. Has to be
    fixed in the storage in the future.
    
    Related to #4366

See https://jenkins.softwareheritage.org/job/DGQL/job/tests-on-diff/123/ for more details.

functional tests

Build is green

Patch application report for D8310 (id=30030)

Rebasing onto 40c0deba...

Current branch diff-target is up to date.

Changes applied before test

commit e3fa1bbb5cdbb468a56b11ca408b0d4b78f95a90
Author: Jayesh Velayudhan <jayesh@softwareheritage.org>
Date:   Wed Aug 24 17:04:56 2022 +0200

    Add revision log field in the revision type
    
    Storage is returning a list of dicts for the revision log.
    Add a from_dict converter to create the model objects from these dicts
    This is not efficient and adds an extra iteration. Has to be
    fixed in the storage in the future.
    
    Related to #4366

See https://jenkins.softwareheritage.org/job/DGQL/job/tests-on-diff/129/ for more details.

lgtm

Merge request was accepted

approved this merge request

Some references in the commit message have been migrated:

• T4366 is now #4366 (closed)

fix typo

Build is green

Patch application report for D8310 (id=30057)

Rebasing onto 45582e79...

Current branch diff-target is up to date.

Changes applied before test

commit a31419d74c4562f3f9d4dad02a7b8ad1590aa97b
Author: Jayesh Velayudhan <jayesh@softwareheritage.org>
Date:   Wed Aug 24 17:04:56 2022 +0200

    Add revision log field in the revision type
    
    Storage is returning a list of dicts for the revision log.
    Add a from_dict converter to create the model objects from these dicts
    This is not efficient and adds an extra iteration. Has to be
    fixed in the storage in the future.
    
    Related to #4366

See https://jenkins.softwareheritage.org/job/DGQL/job/tests-on-diff/132/ for more details.

Merge request was merged

closed