jenkins-bot has submitted this change and it was merged.
Change subject: Remove oresc_is_predicted = 1 in db queries
......................................................................
Remove oresc_is_predicted = 1 in db queries
It doesn't serve any meaningful purpose in this case
Cherry-picked from commit 1265ceb8.
Change-Id: Ie1a7321c2075c1c6fa5b302af52c3ca2569cb8e2
---
M includes/Hooks.php
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Dereckson: Looks good to me, approved
jenkins-bot: Verified
diff --git a/includes/Hooks.php b/includes/Hooks.php
index 5f593bc..9f194b0 100644
--- a/includes/Hooks.php
+++ b/includes/Hooks.php
@@ -123,7 +123,7 @@
$join_conds['ores_classification'] = [ 'LEFT JOIN',
'rc_this_oldid = oresc_rev ' .
- 'AND oresc_is_predicted = 1 AND oresc_class = 1' ];
+ 'AND oresc_class = 1' ];
$join_conds['ores_model'] = [ 'LEFT JOIN',
'oresc_model = oresm_id ' .
@@ -134,7 +134,7 @@
// Override the join conditions.
$join_conds['ores_classification'] = [ 'INNER JOIN',
'rc_this_oldid = oresc_rev ' .
- 'AND oresc_is_predicted = 1 AND oresc_class =
1' ];
+ 'AND oresc_class = 1' ];
// Filter out non-damaging edits.
$conds[] = 'oresc_probability > '
--
To view, visit https://gerrit.wikimedia.org/r/297916
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie1a7321c2075c1c6fa5b302af52c3ca2569cb8e2
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/ORES
Gerrit-Branch: wmf/1.28.0-wmf.9
Gerrit-Owner: Ladsgroup <[email protected]>
Gerrit-Reviewer: Dereckson <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
