Ori Liel has posted comments on this change.
Change subject: restapi: Add "All-Content" Header.
......................................................................
Patch Set 2: (4 inline comments)
....................................................
File
backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendAsyncStatusResource.java
Line 53: }
Line 54:
Line 55: @Override
Line 56: protected R doPopulate(R model, List entity) {
Line 57: return model;
This was impossible because this class is not abstract. But following your
comment, I took the opportunity and made the class abstract. Following that, I
could delegate doPopulate() to predecessors.
Line 58: }
....................................................
File
backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostNicResource.java
Line 134: params.setCheckConnectivity(nic.isCheckConnectivity());
Line 135: }
Line 136: performAction(VdcActionType.UpdateNetworkToVdsInterface,
params);
Line 137:
Line 138: return parent.lookupNic(id, true);
requires out-of-scope refactoring, will be done in another patch
Line 139: }
Line 140:
Line 141: private org.ovirt.engine.core.common.businessentities.Network
getNewNetwork(HostNIC nic) {
Line 142: org.ovirt.engine.core.common.businessentities.Network
newNetwork = null;
....................................................
File
backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostNicsResource.java
Line 100: public HostNicResource getHostNicSubResource(String id) {
Line 101: return inject(new BackendHostNicResource(id, this));
Line 102: }
Line 103:
Line 104: public HostNIC lookupNic(String id, boolean forcePopulate) {
requires out-of-scope refactoring; will be done in another patch
Line 105: List<VdsNetworkInterface> ifaces = getCollection();
Line 106: for (VdsNetworkInterface iface : ifaces) {
Line 107: if (iface.getId().toString().equals(id)) {
Line 108: HostNIC hostNic = map(iface, ifaces);
....................................................
File
backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendReadOnlyDeviceResource.java
Line 43: return collection;
Line 44: }
Line 45:
Line 46: @Override
Line 47: protected D doPopulate(D model, Q entity) {
This is not an abstract class, and should not be abstract (it's generic, and
creating a new instance of it is legal and happens in our code today).
Therefore, delegating to predecessors is not an option
Line 48: return model;
Line 49: }
--
To view, visit http://gerrit.ovirt.org/9815
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I2b0faf2843ec368fafd5270dc23d711d0710bd48
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ori Liel <[email protected]>
Gerrit-Reviewer: Michael Pasternak <[email protected]>
Gerrit-Reviewer: Ori Liel <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
