Changeset: ece85c5d8d31 for MonetDB
URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=ece85c5d8d31
Modified Files:
        clients/iotapi/src/WebSockets/websockets.py
        clients/iotclient/src/Streams/datatypes.py
Branch: iot
Log Message:

Small corrections


diffs (104 lines):

diff --git a/clients/iotapi/src/WebSockets/websockets.py 
b/clients/iotapi/src/WebSockets/websockets.py
--- a/clients/iotapi/src/WebSockets/websockets.py
+++ b/clients/iotapi/src/WebSockets/websockets.py
@@ -30,7 +30,7 @@ def unsubscribe_removed_streams(concaten
             client.remove_subscribed_stream(name)
     WebClientsLock.release()
     for name in concatenated_names:
-        add_log(20, ''.join(['Stream removed: ', name]))
+        add_log(20, ''.join(['Stream ', name, ' removed']))
 
 
 class IOTAPI(WebSocket):
@@ -81,7 +81,7 @@ class IOTAPI(WebSocket):
         self._subscriptions[concatenated_name] = stream
         self._locker.release()
         self.sendMessage({"subscribed": "Subscribed to " + concatenated_name})
-        add_log(20, ''.join(['Client ', self.address[0], 'subscribed stream ', 
concatenated_name]))
+        add_log(20, ''.join(['Client ', self.address[0], 'subscribed to stream 
', concatenated_name]))
 
     def unsubscribe(self, concatenated_name):
         self._locker.acquire_write()
@@ -92,27 +92,31 @@ class IOTAPI(WebSocket):
             del self._subscriptions[concatenated_name]
             self._locker.release()
             self.sendMessage({"unsubscribed": "Unsubscribed to " + 
concatenated_name})
-            add_log(20, ''.join(['Client ', self.address[0], ' unsubscribed 
stream ', concatenated_name]))
+            add_log(20, ''.join(['Client ', self.address[0], ' unsubscribed to 
stream ', concatenated_name]))
 
     def remove_subscribed_stream(self, concatenated_name):
         self._locker.acquire_write()
         if concatenated_name in self._subscriptions:
             del self._subscriptions[concatenated_name]
         self._locker.release()
-        self.sendMessage({"removed": "Stream removed from context: " + 
concatenated_name})
+        self.sendMessage({"removed": 'Stream ' + concatenated_name + ' removed 
from context'})
 
     def send_notification_message(self, concatenated_name, schema_name, 
stream_name, count):
         self._locker.acquire_read()
         if concatenated_name in self._subscriptions:
             self._locker.release()
             self.sendMessage({'notification': {'schema': schema_name, 
'stream': stream_name, 'tuples': count}})
-            add_log(20, ''.join(['Stream notification sent to client ', 
self.address[0]]))
+            add_log(20, ''.join(['Stream ', concatenated_name, ' notification 
sent to client ', self.address[0]]))
         else:
             self._locker.release()
 
     def read_stream_batch(self, concatenated_name, basket_number, limit, 
offset):
-        stream = Streams_context.get_existing_stream(concatenated_name)
-        self.sendMessage(stream.read_tuples(basket_number, limit, offset))
+        try:
+            stream = Streams_context.get_existing_stream(concatenated_name)
+            self.sendMessage(stream.read_tuples(basket_number, limit, offset))
+        except BaseException as ex:
+            self.sendMessage({"error": ex})
+            add_log(50, ex)
 
 
 def init_websockets(host, port):
diff --git a/clients/iotclient/src/Streams/datatypes.py 
b/clients/iotclient/src/Streams/datatypes.py
--- a/clients/iotclient/src/Streams/datatypes.py
+++ b/clients/iotclient/src/Streams/datatypes.py
@@ -288,16 +288,14 @@ class INetType(StreamDataType):
 
         if entry == self.get_nullable_constant():
             array[7] = 1
-            return array
-
-        components = re.split(r'[./]+', entry)
-        for i in range(4):
-            array[i] = int(components[i])
-        if len(components) > 4:  # if it has a mask add it to the array
-            array[4] = int(components[4])
         else:
-            array[4] = 32
-
+            components = re.split(r'[./]+', entry)
+            for i in xrange(4):
+                array[i] = int(components[i])
+            if len(components) > 4:  # if it has a mask add it to the array
+                array[4] = int(components[4])
+            else:
+                array[4] = 32
         return array
 
     def pack_parsed_values(self, extracted_values, counter, parameters):
@@ -323,7 +321,7 @@ class UUIDType(StreamDataType):
         j = 0
         s = 0
 
-        for i in range(16):
+        for i in xrange(16):
             if j in (8, 12, 16, 20):  # do nothing with the dashes
                 s += 1
 
@@ -338,8 +336,8 @@ class UUIDType(StreamDataType):
             s += 1
             j += 1
             array[i] <<= 4
+
             next_char = ord(entry[s])
-
             if 48 <= next_char <= 57:  # between '0' and '9'
                 array[i] |= next_char - 48
             elif 97 <= next_char <= 102:  # between 'a' and 'f'
_______________________________________________
checkin-list mailing list
checkin-list@monetdb.org
https://www.monetdb.org/mailman/listinfo/checkin-list

Reply via email to