Merge pull request #495 from nobodyiam/optimize-client-logerror

remove unnessary Tracer.logError

apollo-client/src/main/java/com/ctrip/framework/apollo/internals/ConfigServiceLocator.java

@@ -122,7 +122,6 @@ public class ConfigServiceLocator implements Initializable {
         logConfigServicesToCat(services);
         return;
       } catch (Throwable ex) {
-        Tracer.logError(ex);
         transaction.setStatus(ex);
         exception = ex;
       } finally {

apollo-client/src/main/java/com/ctrip/framework/apollo/internals/LocalFileConfigRepository.java

@@ -141,7 +141,6 @@ public class LocalFileConfigRepository extends AbstractConfigRepository
       m_fileProperties = this.loadFromLocalCacheFile(m_baseDir, m_namespace);
       transaction.setStatus(Transaction.SUCCESS);
     } catch (Throwable ex) {
-      Tracer.logError(ex);
       transaction.setStatus(ex);
       exception = ex;
       //ignore

apollo-client/src/main/java/com/ctrip/framework/apollo/internals/RemoteConfigLongPollService.java

@@ -180,7 +180,6 @@ public class RemoteConfigLongPollService implements Initializable {
         transaction.setStatus(Transaction.SUCCESS);
       } catch (Throwable ex) {
         lastServiceDto = null;
-        Tracer.logError(ex);
         transaction.setStatus(ex);
         long sleepTimeInSecond = m_longPollFailSchedulePolicyInSecond.fail();
         logger.warn(

apollo-client/src/main/java/com/ctrip/framework/apollo/internals/RemoteConfigRepository.java

@@ -213,11 +213,9 @@ public class RemoteConfigRepository extends AbstractConfigRepository {
             statusCodeException = new ApolloConfigStatusCodeException(ex.getStatusCode(),
                 message);
           }
-          Tracer.logError(statusCodeException);
           transaction.setStatus(statusCodeException);
           exception = statusCodeException;
         } catch (Throwable ex) {
-          Tracer.logError(ex);
           transaction.setStatus(ex);
           exception = ex;
         } finally {