[07:26:11] (CR) jerkins-bot: [V: -1] build: Updating lodash to 4.17.15 [extensions/SemanticSifter] - https://gerrit.wikimedia.org/r/562430 (owner: Libraryupgrader) [10:15:15] (CR) Daimona Eaytoy: [C: -1] Use DB_MASTER for creating temporary database tables (1 comment) [extensions/SemanticDrilldown] - https://gerrit.wikimedia.org/r/562361 (owner: Pastakhov) [10:17:57] (CR) Daimona Eaytoy: [C: -1] "(Note: my comment applies to SELECT queries. The ones which explicitly create a temporary table, i.e. CREATE queries, should indeed be exe" [extensions/SemanticDrilldown] - https://gerrit.wikimedia.org/r/562361 (owner: Pastakhov) [12:02:44] (CR) Pastakhov: "I think it makes no sense to create temporary tables on master database and then try to SELECT from ones on SLAVE database, right?" [extensions/SemanticDrilldown] - https://gerrit.wikimedia.org/r/562361 (owner: Pastakhov) [16:02:31] (CR) Daimona Eaytoy: "> I think it makes no sense to create temporary tables on master" [extensions/SemanticDrilldown] - https://gerrit.wikimedia.org/r/562361 (owner: Pastakhov) [21:36:39] (CR) Yaron Koren: [C: +2] "Sorry for the delay, and thank you!" [extensions/SemanticDrilldown] - https://gerrit.wikimedia.org/r/562361 (owner: Pastakhov) [21:37:46] (PS5) Yaron Koren: Use DB_MASTER for creating temporary database tables [extensions/SemanticDrilldown] - https://gerrit.wikimedia.org/r/562361 (owner: Pastakhov)