Vered Volansky has posted comments on this change.
Change subject: core: DiskImage Hierarchy cleanup
......................................................................
Patch Set 2: (5 inline comments)
....................................................
File
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/DiskImage.java
Line 11: private static final long serialVersionUID = 1533416252250153306L;
Line 12:
Line 13: private Date lastModifiedDate;
Line 14: private ArrayList<String> storagesNames;
Line 15: // TODO comes from DiskImageDynamic
I suppose, will remove.
Line 16: private long actualSizeFromDiskImageDynamic;
Line 17: // TODO comes from DiskImageDynamic
Line 18: private int readRateFromDiskImageDynamic;
Line 19: // TODO comes from DiskImageDynamic
Line 117: public Boolean getActive() {
Line 118: return getImage().isActive();
Line 119: }
Line 120:
Line 121: public void setActive(Boolean active) {
Done
Line 122: getImage().setActive(Boolean.TRUE.equals(active));
Line 123: }
Line 124:
Line 125: @Override
Line 272: public ArrayList<String> getStoragesNames() {
Line 273: return storagesNames;
Line 274: }
Line 275:
Line 276: public void setStoragesNames(ArrayList<String> storagesNames) {
Done, will also change in all other places. Notice on re-review.
Line 277: this.storagesNames = storagesNames;
Line 278: }
Line 279:
Line 280: @Deprecated
Line 277: this.storagesNames = storagesNames;
Line 278: }
Line 279:
Line 280: @Deprecated
Line 281: public Guid getimage_group_id() {
Deprecated, as is the next. All untouched methods in this patch are deprecated.
Line 282: return getId();
Line 283: }
Line 284:
Line 285: @Deprecated
....................................................
File
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/IImage.java
Line 23: void setDescription(String value);
Line 24:
Line 25: Guid getItGuid();
Line 26:
Line 27: void setItGuid(Guid value);
Feel free to suggest other names.
Line 28:
Line 29: int getReadRateKbPerSec();
Line 30:
Line 31: void setReadRateKbPerSec(int value);
--
To view, visit http://gerrit.ovirt.org/12051
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaf49ef042266187a6122ae4a667afd16d80efa52
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: Tal Nisan <tni...@redhat.com>
Gerrit-Reviewer: Vered Volansky <vvola...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
