shakedtom / mycheckpoint

Automatically exported from code.google.com/p/mycheckpoint
0 stars 1 forks source link

Auto-deploy is unnecessarily verbose #4

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago

As stated in the release notes:
"
mycheckpoint now has a version recognition mechanism. There is no need to call 
mycheckpoint 
with the “deploy” argument on first install or after upgrade. mycheckpoint 
will recognize a 
change of version and will auto-deploy before moving on to monitoring your 
system."

This is great, although I don't quite like how it is so verbose. 
Perhaps avoid printing out that the table doesn't exist (i.e.: 

/usr/bin/mycheckpoint:109: Warning: Unknown table 'metadata'
  num_affected_rows = cursor.execute(query)

)

This is of course from the DROP TABLE IF EXISTS..  Perhaps a create table if 
not exists instead? 
Or do I have the wrong idea and you prefer having verbose output as default? 

Original issue reported on code.google.com by NWesterl...@gmail.com on 7 Jan 2010 at 2:48

GoogleCodeExporter commented 8 years ago
Indeed, the warnings are unnecessarily verbose. I will see that they are 
removed.

Original comment by shlomi.n...@gmail.com on 12 Jan 2010 at 10:35

GoogleCodeExporter commented 8 years ago
Fixed in Revision: r90.
Now MySQLdb warnings are suppressed

Original comment by shlomi.n...@gmail.com on 14 Feb 2010 at 6:22