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