Synerise / android-sdk

Synerise SDK for Android
9 stars 8 forks source link

'eventCreateTime' semantic has changed #3

Closed marecik16 closed 4 years ago

marecik16 commented 4 years ago

Yesterday I upgraded SDK from 3.3.13 to 3.6.6.

I was supprised, that the semantic of 'eventCreateTime' has changed. In the previous version, 'eventCreateTime' meant time, when the event was ~registered in the System and it was slightly later than event's reference 'time'. In particular, when a mobile had no access to the internet, the difference between 'time' and 'eventCreateTime' meant (more or less) the time duration without internet connection. That behaviour helps me track when the events were really sent, especially if I group them in batches and do not flush instantly.

In the new version 'eventCreateTime' is EXACTLY the same as event's 'time' and it seems to be completely redundant.

Is it a bug or an intented change?

konned commented 4 years ago

It was an intended change due to inconsistency in data.