Address comments

Author: minggangw

lib/service.js

@@ -17,6 +17,7 @@
 const rclnodejs = require('./native_loader.js');
 const DistroUtils = require('./distro.js');
 const Entity = require('./entity.js');
+const Logging = require('./logging.js');
 const debug = require('debug')('rclnodejs:service');
 
 /**
@@ -88,7 +89,8 @@ class Service extends Entity {
         );
       })
       .catch((error) => {
-        debug(`Error processing ${this._serviceName} request: ${error}`);
+        const logger = Logging.getLogger('rclnodejs');
+        logger.error(`Error processing ${this._serviceName} request: ${error}`);
       });
   }
 

lib/time_source.js

@@ -73,6 +73,9 @@ class TimeSource {
     });
     if (enabled) {
       this._subscribeToClockTopic();
+    } else if (this._node && this._clockSubscription) {
+      this._node.destroySubscription(this._clockSubscription);
+      this._clockSubscription = undefined;
     }
   }