jenkins-bot has submitted this change and it was merged.
Change subject: Map JSON Schema 'integer' type to MySQL BIGINTs rather than INTs
......................................................................
Map JSON Schema 'integer' type to MySQL BIGINTs rather than INTs
As Brion pointed out, with revision IDs already in the 600m range, we can
expect to need to log numbers outside the range of INTs (+/- 2.1b).
Change-Id: Ice7ea83828bb023af17f943b680f933a805c4233
---
M server/eventlogging/jrm.py
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Ori.livneh: Looks good to me, approved
Springle: Looks good to me, but someone else must approve
jenkins-bot: Verified
diff --git a/server/eventlogging/jrm.py b/server/eventlogging/jrm.py
index c493685..02491a6 100644
--- a/server/eventlogging/jrm.py
+++ b/server/eventlogging/jrm.py
@@ -84,7 +84,7 @@
mappers = collections.OrderedDict((
('type', {
'boolean': {'type_': sqlalchemy.Boolean},
- 'integer': {'type_': sqlalchemy.Integer},
+ 'integer': {'type_': sqlalchemy.BigInteger},
'number': {'type_': sqlalchemy.Float},
'string': {'type_': sqlalchemy.Unicode(STRING_MAX_LEN)},
}),
--
To view, visit https://gerrit.wikimedia.org/r/168003
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ice7ea83828bb023af17f943b680f933a805c4233
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/EventLogging
Gerrit-Branch: master
Gerrit-Owner: Ori.livneh <[email protected]>
Gerrit-Reviewer: Brion VIBBER <[email protected]>
Gerrit-Reviewer: Ori.livneh <[email protected]>
Gerrit-Reviewer: Springle <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
