Milimetric has submitted this change and it was merged.

Change subject: Split up queries to avoid MariaDB limit
......................................................................


Split up queries to avoid MariaDB limit

MariaDB does not let you join more than 61 tables.  This is a very
sensible limit as you should probably never want to join so many tables.
This should be fixed by using a better report runner that repeats simple
queries once for each wiki.  Marcel's reportupdater does just that, and
we should look into using it.  However, until then we must make sure
this query keeps running so we use this terrible hack.  The query must
keep running because if it doesn't we lose data (since the tables being
queried change over time without storing the changes).

Bug: T98532
Change-Id: I29051d6d797b32a33b952840c75aaa7804b042a8
---
M language/config.yaml
M language/content_translation_beta.sql
2 files changed, 14 insertions(+), 12 deletions(-)

Approvals:
  Milimetric: Looks good to me, approved



diff --git a/language/config.yaml b/language/config.yaml
index 0672ca1..2d5fd6a 100644
--- a/language/config.yaml
+++ b/language/config.yaml
@@ -13,14 +13,15 @@
 defaults:
     db: el
 # the key in the graph section is the name of the sql file and generated file
-graphs:
-    content_translation_beta:
-        title: 'People enabling Content Translation'
-        frequency: hourly
-        timeboxed: true
-        starts: 2015-01-13
-    content_translation_beta_manual:
-        title: 'People manually enabling Content Translation'
-        frequency: hourly
-        timeboxed: true
-        starts: 2015-01-13
+# NOTE: these don't run properly, so they should stay disabled until 
generate.py is fixed properly:
+#graphs:
+    #content_translation_beta:
+        #title: 'People enabling Content Translation'
+        #frequency: hourly
+        #timeboxed: true
+        #starts: 2015-01-13
+    #content_translation_beta_manual:
+        #title: 'People manually enabling Content Translation'
+        #frequency: hourly
+        #timeboxed: true
+        #starts: 2015-01-13
diff --git a/language/content_translation_beta.sql 
b/language/content_translation_beta.sql
index da4b634..f545feb 100644
--- a/language/content_translation_beta.sql
+++ b/language/content_translation_beta.sql
@@ -29,7 +29,8 @@
        cvwiki,
        euwiki,
        glwiki,
-       koiwiki,
+       koiwiki
+
        kvwiki,
        lezwiki,
        ocwiki,

-- 
To view, visit https://gerrit.wikimedia.org/r/209666
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I29051d6d797b32a33b952840c75aaa7804b042a8
Gerrit-PatchSet: 1
Gerrit-Project: analytics/limn-language-data
Gerrit-Branch: master
Gerrit-Owner: Milimetric <dandree...@wikimedia.org>
Gerrit-Reviewer: Milimetric <dandree...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to