On Wed, Aug 27, 2014 at 10:59 PM, Alvaro Herrera <alvhe...@2ndquadrant.com> wrote:
> Not necessarily, because it's harmless. It's there for purely > aesthetical reasons, so it's your choice whether to add it or not. > Having it there is slightly easier on somebody reading the code, > perhaps. On my side, that's up to you Fujii-san. The patch does what it states, I only think that this extra 0 should be added either everywhere or nowhere. Not mandatory either: drop test_param_unit in the regression tests after running the test queries. Regards, -- Michael