Yaniv Dary has posted comments on this change.

Change subject: history: sync users data for user usage tables
......................................................................


Patch Set 11:

(1 comment)

http://gerrit.ovirt.org/#/c/27338/11/etl_export/src/ovirt_engine_dwh/aggregationtohourly_3_5/AggregationToHourly.java
File 
etl_export/src/ovirt_engine_dwh/aggregationtohourly_3_5/AggregationToHourly.java:

Line 23773
Line 23774
Line 23775
Line 23776
Line 23777
You removed this column and now you don't reference it in group. What will 
happen to existing data? without this in group it will be done incorrectly.


-- 
To view, visit http://gerrit.ovirt.org/27338
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id236a3112947e00a58175f0361d44afda692b894
Gerrit-PatchSet: 11
Gerrit-Project: ovirt-dwh
Gerrit-Branch: master
Gerrit-Owner: Shirly Radco <sra...@redhat.com>
Gerrit-Reviewer: Shirly Radco <sra...@redhat.com>
Gerrit-Reviewer: Yaniv Dary <yd...@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