Should dialect:mssql have an effect on natural joins?

[spenserblack]

This is a really awesome tool!

Starting as a question as I don't know if this is intended behavior or not.

prql dialect:mssql
from employees
join jobs [employee_id]

has the output below (playground and prql-compiler 0.2.1)

SELECT
  employees.*,
  jobs.*,
  employee_id
FROM
  employees
  JOIN jobs USING(employee_id)

As far as I know (just based on usage and some Googling), USING(employee_id) isn't valid in SQL Server. So, when using dialect:mssql, should the compiler

• have an error that this syntax isn't available for dialect:mssql (and suggest [employees.employee_id==jobs.employee_id])?
• have a warning that this might not work with the given dialect?
• change resulting SQL from USING(employee_id) to ON employees.employee_id = jobs.employee_id?

[aljazerzen]

Hello, thank you for pointing this out and sorry for late response.

Fortunately, we are changing behavior for all dialects from USING(x) to ON(left.x = right.x) in an upcoming release.

We are (well, at least I am) more focused on postgres dialect, only because it's the one I'm familiar with. If you find any more mismatches between dialects, please do report them.