Martin Peřina has posted comments on this change.

Change subject: engine: make EngineSession queriable
......................................................................


Patch Set 1:

(1 comment)

https://gerrit.ovirt.org/#/c/38378/1/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/EngineSession.java
File 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/EngineSession.java:

Line 14
Line 15
Line 16
Line 17
Line 18
> If we don't have engineSessionId on UI, how can we get sessionId only with 
Ahh, please ignore my previous comment, we have also id attribute.

In that case, we may change engineSessionId to transient


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9bbdd6e1828b9be5500ec2f998c62511f102deda
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yevgeny Zaspitsky <yzasp...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Martin Peřina <mper...@redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <wallaroo1...@gmail.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