I found projections to be quite useful in the printer output for SelectExpression, I'd strongly prefer to keep it.

Any idea why our [DebuggerDisplay] are always inside #if DEBUG?

Only including DebuggerDisplay on DEBUG builds would limit their use to the EF team. Intentional? It seems weird to improve debugging but not for all developers.

I found projections to be quite useful in the printer output for SelectExpression, I'd strongly prefer to keep it.

OK. I'll leave things as they are in this PR: the default ExpressionPrinter.Print() for SelectExpression will show the full details including projections. For [DebuggerDisplay] directly on SelectExpression, we'll show the short SQL since space there is short and it's harder to read multi-line strings (before this PR there was no [DebuggerDisplay] at all). It's now also always possible to just go into DebugView to get the full string.

Any idea why our [DebuggerDisplay] are always inside #if DEBUG?

Only including DebuggerDisplay on DEBUG builds would limit their use to the EF team. Intentional? It seems weird to improve debugging but not for all developers.

Yep, I've run into this when debugging from the external PostgreSQL provider. I'll remove these.

@maumar I've done a tiny bit more of cleanup/changes, including changing JsonScalarExpression to print col -> path, like the actual JSON lookup operator in all databases except for SQL Server; for JsonQueryExpression i've changed to col Q-> path to distinguish it.

Will let you review all the above before merging in case you prefer to change something.

@roji looking good!