Skip to content

Commit 7f00e54

Browse files
Remove unused logic for handling null entities (#71)
1 parent c4d9cca commit 7f00e54

File tree

1 file changed

+2
-13
lines changed

1 file changed

+2
-13
lines changed

redisgraph/query_result.py

Lines changed: 2 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,8 @@
88
class ResultSetColumnTypes(object):
99
COLUMN_UNKNOWN = 0
1010
COLUMN_SCALAR = 1
11-
COLUMN_NODE = 2
12-
COLUMN_RELATION = 3
11+
COLUMN_NODE = 2 # Unused as of RedisGraph v2.1.0, retained for backwards compatibility.
12+
COLUMN_RELATION = 3 # Unused as of RedisGraph v2.1.0, retained for backwards compatibility.
1313

1414

1515
class ResultSetScalarTypes(object):
@@ -104,10 +104,6 @@ def parse_entity_properties(self, props):
104104
return properties
105105

106106
def parse_node(self, cell):
107-
# Return None if we received a null value.
108-
if self.is_null_scalar(cell):
109-
return None
110-
111107
# Node ID (integer),
112108
# [label string offset (integer)],
113109
# [[name, value type, value] X N]
@@ -120,10 +116,6 @@ def parse_node(self, cell):
120116
return Node(node_id=node_id, label=label, properties=properties)
121117

122118
def parse_edge(self, cell):
123-
# Return None if we received a null value.
124-
if self.is_null_scalar(cell):
125-
return None
126-
127119
# Edge ID (integer),
128120
# reltype string offset (integer),
129121
# src node ID offset (integer),
@@ -227,9 +219,6 @@ def pretty_print(self):
227219
def is_empty(self):
228220
return len(self.result_set) == 0
229221

230-
def is_null_scalar(self, cell):
231-
return cell == [ResultSetScalarTypes.VALUE_NULL, None]
232-
233222
@staticmethod
234223
def _get_value(prop, statistics):
235224
for stat in statistics:

0 commit comments

Comments
 (0)