v16 beautifier adds space after OVER

[Martins]

v16 (16.0.3.2156) adds a space after the keyword "OVER". Is it a feature or a bug?

v15
SELECT 1 OVER(ORDER BY 1)
  FROM dual;

v16
SELECT 1 OVER (ORDER BY 1)
  FROM dual;

Now with v16 we see this unnecessary code change whenever we use beautifier (and we use it ALWAYS before deploying)

 

[Marco Kalter]

We'll look into this.

 

[William Robertson]

I agree that you should have the choice, and unexpected changes complicate change history and pull requests etc, but not having a space there seems a rather odd preference to me.

Normally you would have no space between a function and its arguments, like

nvl(somevalue, 0)

and not

nvl (somevalue, 0)

but you would normally have a space before a SQL expression, like

and product_type in ('Cheese', 'Jam')

I've seen this type of rule enforced in Java teams, for example.

Then again, your keywords are in uppercase and 'SELECT' and 'FROM' don't line up so what can I say

 

[Martins]

My SELECT and FROM do line up, just to the right

Anyway, I'm just bringing attention to this - if it was an unintentional change then maybe it is worth looking into what other unintentional changes there could be.