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

minor cleanups in ezdfs cluster file handler

    XMLWordPrintable

Details

    • Icon: Improvement Improvement
    • Resolution: Obsolete
    • Icon: Medium Medium
    • None
    • 4.2.0, 4.3.0, 4.4.0beta1
    • Legacy > Clustering
    • None

    Description

      1. in function _abortCacheGeneration a useless db transaction is used, since there is no rollback possible and only a single db update. Is this wanted?

      2. in function _purgeByLike, the rollback on line 283 of mysql.php is a bug, as if there are nested transactions (which the driver supports), the code will rollback existing outer transactions. Using a commit is recommended here, to simply release the lock on the lines held with the FOR UPDATE select

      3. in the same function, the select on line 297 risks to not see the same lines selected on line 271, if another process comes in with a commit in between and mysql transaction isolation level allows it. We should instead get a list of file names on 1st select, and use an IN() clause in the 2nd select using that list

      4. function _fetch does not define $fname to pass on to _fetchmetadata

      5. same for _rename

      6. in function _report, static $numQueries should be replaced with self::$numQueries

      Attachments

        Activity

          People

            unknown unknown
            72f8acac-185f-4a54-9470-a7473f50daab@accounts.ibexa.co Gaetano Giunta
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated: