Skip to content

Commit e9e4459

Browse files
Copilotmykaul
andcommitted
Refactor decode_val to use if/else instead of ternary operators
Per review feedback, simplified decode_val function structure: - Use single 'if uses_ce:' clause for encrypted column path - Else clause simply returns col_md[3].from_binary(val, protocol_version) - More readable than chained ternary operators Maintains same functionality while improving code clarity. Co-authored-by: mykaul <[email protected]>
1 parent 8cda794 commit e9e4459

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

cassandra/protocol.py

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -726,9 +726,12 @@ def recv_results_rows(self, f, protocol_version, user_type_map, result_metadata,
726726

727727
def decode_val(val, col_md, col_desc):
728728
uses_ce = column_encryption_policy.contains_column(col_desc)
729-
col_type = column_encryption_policy.column_type(col_desc) if uses_ce else col_md[3]
730-
raw_bytes = column_encryption_policy.decrypt(col_desc, val) if uses_ce else val
731-
return col_type.from_binary(raw_bytes, protocol_version)
729+
if uses_ce:
730+
col_type = column_encryption_policy.column_type(col_desc)
731+
raw_bytes = column_encryption_policy.decrypt(col_desc, val)
732+
return col_type.from_binary(raw_bytes, protocol_version)
733+
else:
734+
return col_md[3].from_binary(val, protocol_version)
732735

733736
def decode_row(row):
734737
return tuple(decode_val(val, col_md, col_desc) for val, col_md, col_desc in zip(row, column_metadata, col_descs))

0 commit comments

Comments
 (0)