Last modified: 2013-06-14 18:26:47 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T51586, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 49586 - VisualEditor: EventLogging gives the same pageViewSessionId for all edits
VisualEditor: EventLogging gives the same pageViewSessionId for all edits
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
General (Other open bugs)
unspecified
All All
: Highest normal
: VE-deploy-2013-06-13
Assigned To: James Forrester
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-06-14 17:46 UTC by James Forrester
Modified: 2013-06-14 18:26 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description James Forrester 2013-06-14 17:46:08 UTC

    
Comment 1 James Forrester 2013-06-14 17:48:31 UTC
All events in Edit_5563071 appear to have the same value for event_pageViewSessionId (2147483647).

When fixing, we will want to bump the event_version value.
Comment 2 Ori Livneh 2013-06-14 17:50:57 UTC
(In reply to comment #1)
> All events in Edit_5563071 appear to have the same value for
> event_pageViewSessionId (2147483647).

It was simply overflowing its column type (int). I set it to 'bigint' and it's fine now.

It's odd that we haven't run into this before! It's because we ordinarily convert timestamps to byte strings, like the rest of MediaWiki. This is the first time we've tried to save timestamps as integers. The int type is good for values between +/- 2,147,483,647, which has been adequate for all other use cases.

> When fixing, we will want to bump the event_version value.

Well, since this did not require a deployment, I instead moved all existing rows in the 'Edit' table to 'z_Edit_5563071'. Any events that go into the current 'Edit_5563071' will be fine.
Comment 3 Gerrit Notification Bot 2013-06-14 18:19:47 UTC
Related URL: https://gerrit.wikimedia.org/r/68712 (Gerrit Change Iddb9eb9c64b55b25445ddf9e474d312b685978a6)
Comment 4 James Forrester 2013-06-14 18:26:47 UTC
Merged and we'll get this out on Monday so we can test it.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links