Обсуждение: getting carriage return character in vacuumo

Hi,

While using the vacuumlo utility I encountered a redundant carriage return(\r') character in the output. It is required in any scenario? If not, please find attached a tiny patch which will get rid of that extra '\r' character.

Regards,
Usama

Muhammad Usama <m.usama@gmail.com> writes:
> While using the vacuumlo utility I encountered a redundant carriage
> return(\r') character in the output. It is required in any scenario? If not,
> please find attached a tiny patch which will get rid of that extra '\r'
> character.

I think the idea there is to overwrite successive progress messages
on the same line.  It's maybe not going to work in all environments,
though, so perhaps we should reconsider that bit of cuteness.
        regards, tom lane

On Fri, Apr 08, 2011 at 02:10:56PM -0400, Tom Lane wrote:
> Muhammad Usama <m.usama@gmail.com> writes:
> > While using the vacuumlo utility I encountered a redundant carriage
> > return(\r') character in the output. It is required in any scenario? If not,
> > please find attached a tiny patch which will get rid of that extra '\r'
> > character.
>
> I think the idea there is to overwrite successive progress messages
> on the same line.  It's maybe not going to work in all environments,
> though, so perhaps we should reconsider that bit of cuteness.

One could hinge it on batch or non-batch mode, the default being
determined by whether it's running in a terminal or not.

Have a nice day,
--
Martijn van Oosterhout   <kleptog@svana.org>   http://svana.org/kleptog/
> Patriotism is when love of your own people comes first; nationalism,
> when hate for people other than your own comes first.
>                                       - Charles de Gaulle

Martijn van Oosterhout wrote:
-- Start of PGP signed section.
> On Fri, Apr 08, 2011 at 02:10:56PM -0400, Tom Lane wrote:
> > Muhammad Usama <m.usama@gmail.com> writes:
> > > While using the vacuumlo utility I encountered a redundant carriage
> > > return(\r') character in the output. It is required in any scenario? If not,
> > > please find attached a tiny patch which will get rid of that extra '\r'
> > > character.
> > 
> > I think the idea there is to overwrite successive progress messages
> > on the same line.  It's maybe not going to work in all environments,
> > though, so perhaps we should reconsider that bit of cuteness.
> 
> One could hinge it on batch or non-batch mode, the default being
> determined by whether it's running in a terminal or not.

pg_upgrade also uses this \r status update trick unconfitionally. 
Should we fix these?

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + It's impossible for everything to be true. +