[DS-3849] REST API items resource returns items in non-deterministic order (further 6.x changes) #2501
Conversation
tdonohue
added
bug
port to main
There was a problem hiding this comment.
👍 Changes seem fine to me. However, both this PR and the original #2460 PR must also be applied to master. Otherwise, these fixes will not be carried forward (which is obviously not ideal). Please also create a PR to apply both of these to our master branch. Thanks!
kshepherd
changed the title
There was a problem hiding this comment.
+1
Merging this so it makes it in for 6.4 - the master port #2515 is still failing some tests (seems to be seeing Public Item 3 where it expects Public Item 2 -- and I assume the fault is with the tests?) but this code looks good and behaves well in 6.x. Thanks @PhilipVis
|
Actually - first, I'm just going to Close and re-open this PR so we can trigger the Travis CI build again which seemed to stall, and is too old for me to find in the Travis build list |
[DS-3849] REST API items resource returns items in non-deterministic order (further 6.x changes)
tdonohue
removed
the
port to main
Continuation of the work in #2460.
This PR updates the ItemDAOImpl class to make sure all item lists returned by this class will now be in a deterministic order.
Jira Issue: https://jira.duraspace.org/browse/DS-3849