Allon Mureinik has submitted this change and it was merged. Change subject: core: Use longs to represent dates in QueryData ......................................................................
core: Use longs to represent dates in QueryData Use longs representing the amount of milliseconds since the epoch instead of Data instances in order to prevent accidental change of QueryData's internal members externally. Besides being a good defensive practice, this patch solves the EI_EXPOSE_REP and EI_EXPOSE_REP2 FindBugs warnings on this class. Change-Id: I9eb83c123a4fd9a1b2abc3bdb74d91a1391245d2 Signed-off-by: Allon Mureinik <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/QueryData.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/SearchQuery.java 2 files changed, 5 insertions(+), 7 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22144 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9eb83c123a4fd9a1b2abc3bdb74d91a1391245d2 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <[email protected]> Gerrit-Reviewer: Alissa Bonas <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
