Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

False positive Multiple columns found with name coachTagId #560

Closed
vanniktech opened this issue Sep 4, 2023 · 0 comments · Fixed by #574
Closed

False positive Multiple columns found with name coachTagId #560

vanniktech opened this issue Sep 4, 2023 · 0 comments · Fixed by #574
Labels

Comments

@vanniktech
Copy link
Contributor

vanniktech commented Sep 4, 2023

Failing ANSI SQL

SELECT
  coachTagCoachee.coachTagId AS coachTagId,
  coachTag.name AS coachTagName,
  COUNT(coachTag.id) AS numberOfCoachees
  FROM chat
  JOIN coachee
    ON coachee.id = chat.coacheeId
  LEFT JOIN coachTagCoachee
    ON coachTagCoachee.coacheeId = coachee.id
  LEFT JOIN coachTag
    ON coachTag.id = coachTagCoachee.coachTagId
  GROUP BY coachTagId
  ORDER BY coachTagName IS NULL, coachTagName COLLATE NOCASE ASC, coachTagId ASC
;

Description

Fails with:

Screenshot 2023-09-04 at 13 57 13

Full sql to reproduce:

CREATE TABLE coachee (
  id TEXT NOT NULL PRIMARY KEY
);

CREATE TABLE chat (
  id TEXT NOT NULL PRIMARY KEY,
  coacheeId TEXT NOT NULL
);

CREATE TABLE coachTagCoachee (
  coachTagId TEXT NOT NULL,
  coacheeId TEXT NOT NULL,
  PRIMARY KEY (coachTagId, coacheeId)
);

CREATE TABLE coachTag (
  id TEXT NOT NULL PRIMARY KEY,
  name TEXT NOT NULL
);

test:
SELECT
  coachTagCoachee.coachTagId AS coachTagId,
  coachTag.name AS coachTagName,
  COUNT(coachTag.id) AS numberOfCoachees
  FROM chat
  JOIN coachee
    ON coachee.id = chat.coacheeId
  LEFT JOIN coachTagCoachee
    ON coachTagCoachee.coacheeId = coachee.id
  LEFT JOIN coachTag
    ON coachTag.id = coachTagCoachee.coachTagId
  GROUP BY coachTagId
  ORDER BY coachTagName IS NULL, coachTagName COLLATE NOCASE ASC, coachTagId ASC
;
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant