Allon Mureinik has posted comments on this change.

Change subject: core: Remove constant SIZE_IS_NOT_AVAILABLE
......................................................................


Patch Set 2:

(1 comment)

https://gerrit.ovirt.org/#/c/39004/2/backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/RepoFileMetaDataDAOTest.java
File 
backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/RepoFileMetaDataDAOTest.java:

Line 299:     private static RepoImage getNewIsoRepoFile() {
Line 300:         RepoImage newRepoFileMap = new RepoImage();
Line 301:         newRepoFileMap.setFileType(ImageFileType.ISO);
Line 302:         newRepoFileMap.setRepoImageId("isoDomain.iso");
Line 303:         newRepoFileMap.setLastRefreshed(System.currentTimeMillis());
Should be 0L, since it's now a java.lang.Long
Line 304:         newRepoFileMap.setSize(0);
Line 305:         newRepoFileMap.setDateCreated(null);
Line 306:         
newRepoFileMap.setRepoDomainId(FixturesTool.STORAGE_DOAMIN_NFS_ISO);
Line 307:         return newRepoFileMap;


-- 
To view, visit https://gerrit.ovirt.org/39004
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ida18353f1d5c976ee70e65045b80410dded2bb1f
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Vered Volansky <vvola...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to