Uploaded image for project: 'eZ Publish / Platform'
  1. eZ Publish / Platform
  2. EZP-29790

ContentService::loadContentListByContentInfo loads Content fields in random order on PosgreSQL

    Details

    • Sprint:
      [2.4] - Sprint 5

      Description

      The APIs for ContentService::loadContentListByContentInfo and ContentService::loadContent load list of internal fields without any order, which for PostgreSQL means it is random. This results in randomly failing integration tests. We execute them nightly, so the need to fix becomes important.

      Possible ways to fix:
      1. Introduce proper ordering for both mentioned API.
      2. Do unordered assertion in test which fails (ref.: https://travis-ci.org/ezsystems/ezpublish-kernel/jobs/450304636#L909).

      We should aim at the option 1. since we don't want our system to behave randomly. The option 2. is a workaround in case of a blocker (e.g. related to BC).

        Activity

        Andrzej Longosz created issue -
        Andrzej Longosz made changes -
        Field Original Value New Value
        Status Open [ 1 ] Confirmed [ 10037 ]
        Andrzej Longosz made changes -
        Status Confirmed [ 10037 ] Backlog [ 10000 ]
        Andrzej Longosz made changes -
        Status Backlog [ 10000 ] Specification [ 10002 ]
        Andrzej Longosz made changes -
        Status Specification [ 10002 ] Specification Done [ 10003 ]
        Assignee Andrzej Longosz [ andrzej.longosz@ez.no ]
        Andrzej Longosz made changes -
        Assignee Andrzej Longosz [ andrzej.longosz@ez.no ]
        Andrzej Longosz made changes -
        Summary ContentService::loadContentListByContentInf loads Content fields in random order on PosgreSQL ContentService::loadContentListByContentInfo loads Content fields in random order on PosgreSQL
        Sylvain Guittard made changes -
        Sprint [2.4] - Sprint 4 [ 118 ]
        Sylvain Guittard made changes -
        Sprint [2.4] - Sprint 4 [ 118 ] Candidates for next sprint [ 109 ]
        Sylvain Guittard made changes -
        Rank Ranked lower
        Show
        Andrzej Longosz added a comment - PR: https://github.com/ezsystems/ezpublish-kernel/pull/2491
        Andrzej Longosz made changes -
        Status Specification Done [ 10003 ] Development [ 3 ]
        Andrzej Longosz made changes -
        Status Development [ 3 ] Development Review [ 10006 ]
        Sylvain Guittard made changes -
        Sprint Candidates for next sprint [ 109 ] [2.4] - Sprint 5 [ 119 ]
        Sylvain Guittard made changes -
        Rank Ranked lower
        Hide
        Andrzej Longosz added a comment -

        Already merged into 6.7 as ezsystems/ezpublish-kernel@baed192

        Show
        Andrzej Longosz added a comment - Already merged into 6.7 as ezsystems/ezpublish-kernel@baed192
        Andrzej Longosz made changes -
        Assignee Andrzej Longosz [ andrzej.longosz@ez.no ]
        Status Development Review [ 10006 ] Closed [ 6 ]
        Fix Version/s 1.7.9 [ 15015 ]
        Fix Version/s 1.13.5 [ 15016 ]
        Fix Version/s 2.2.4 [ 15083 ]
        Fix Version/s 2.3.3 [ 15096 ]
        Resolution Fixed [ 1 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Confirmed Confirmed
        9s 1 Andrzej Longosz 06/Nov/18 12:46 PM
        Confirmed Confirmed Backlog Backlog
        15s 1 Andrzej Longosz 06/Nov/18 12:46 PM
        Backlog Backlog Specification Specification
        47s 1 Andrzej Longosz 06/Nov/18 12:47 PM
        Specification Specification Specification Done Specification Done
        8s 1 Andrzej Longosz 06/Nov/18 12:47 PM
        Specification Done Specification Done Development Development
        23d 21h 58m 1 Andrzej Longosz 30/Nov/18 10:46 AM
        Development Development Development Review Development Review
        10s 1 Andrzej Longosz 30/Nov/18 10:46 AM
        Development Review Development Review Closed Closed
        3d 1h 13m 1 Andrzej Longosz 03/Dec/18 12:00 PM

          People

          • Assignee:
            Unassigned
            Reporter:
            Andrzej Longosz
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Agile