Mailing List Archive

[OSSTEST PATCH v2 19/41] Executive: Drop redundant AND clause
In "Executive: Use index for report__find_test" we changed an EXISTS
subquery into a JOIN.

Now, the condition r.flight=f.flight is redundant because this is the
join column (from USING).

No functional change.

CC: George Dunlap <George.Dunlap@citrix.com>
Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
---
Osstest/Executive.pm | 1 -
1 file changed, 1 deletion(-)

diff --git a/Osstest/Executive.pm b/Osstest/Executive.pm
index 684cafc3..2f81e89d 100644
--- a/Osstest/Executive.pm
+++ b/Osstest/Executive.pm
@@ -433,7 +433,6 @@ END
WHERE name=?
AND name LIKE 'revision\_%'
AND val=?
- AND r.flight=f.flight
AND ${\ main_revision_job_cond('r.job') }
END
push @params, "revision_$tree", $revision;
--
2.20.1
Re: [OSSTEST PATCH v2 19/41] Executive: Drop redundant AND clause [ In reply to ]
> On Jul 31, 2020, at 12:37 PM, Ian Jackson <ian.jackson@eu.citrix.com> wrote:
>
> In "Executive: Use index for report__find_test" we changed an EXISTS
> subquery into a JOIN.
>
> Now, the condition r.flight=f.flight is redundant because this is the
> join column (from USING).
>
> No functional change.
>
> CC: George Dunlap <George.Dunlap@citrix.com>
> Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>

Reviewed-by: George Dunlap <george.dunlap@citrix.com>