Uploaded image for project: 'ModeShape'
  1. ModeShape
  2. MODE-2677

BufferingSequence.loadAll() depend on the order of batches

    XMLWordPrintable

Details

    • Bug
    • Resolution: Done
    • Major
    • 5.4.0.Final
    • 5.1.0.Final
    • Query
    • None

    Description

      After updating to the latest version we were facing an ugly bug with the indices.
      After some time debugging and detect where it could come from we found the mistake.

      In commit 9db7a79 hchiorean wanted to shorten the loadAll()-Method from the BufferingSequence.
      But the current code depend now on the order of batches to iterate.

      Example:
      Is the first batch empty (bacth.hasNext() == false) it wouldn't count any batchsize ever.
      Before this change, the "old" while-loop tried all batches until any found to count.

      It could be fixed easily if you change line 140 to firstBatchCounted = batchSize != 0;

      Attachments

        Issue Links

          Activity

            People

              hchiorean Horia Chiorean (Inactive)
              x-out_jira X-out portal (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: