search

ddavisqa / google-refine

Automatically exported from code.google.com/p/google-refine
0 stars 0 forks source link

Patch for /trunk/main/src/com/google/refine/browsing/facets/TextSearchFacet.java #323

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Throwable e.printStackTrace() should be removed ??

Original issue reported on code.google.com by thadguidry on 28 Jan 2011 at 10:57

Attachments:

GoogleCodeExporter commented 9 years ago 
Are you using NetBeans IDE?

I had the same issue. Found the warning under:
Options > Editor > Hints > Code Maturity Issues > Print Stack Trace

It says "This inspection reports any uses Throwable.printStackTrace() without 
arguments. These are often temporary debugging statements, and should probably 
be either removed from production code, or replaced by a more robust logging 
facility."

Original comment by Mer...@gmail.com on 12 Mar 2011 at 9:34

GoogleCodeExporter commented 9 years ago 
I'm going to defer this for now.  If it's going to be fixed, it should be as 
part of a general review of the error handling.  It's better to have the 
information for debugging, even if it's only in the log.

Original comment by tfmorris on 6 Jun 2011 at 10:13