Skip to content

Commit a2896ac

Browse files
committed
rename a parameter and a function
1 parent 6398ecb commit a2896ac

File tree

4 files changed

+14
-14
lines changed

4 files changed

+14
-14
lines changed

Diff for: pymysqlreplication/binlogstream.py

+4-4
Original file line numberDiff line numberDiff line change
@@ -143,7 +143,7 @@ def __init__(self, connection_settings, server_id,
143143
is_mariadb=False,
144144
annotate_rows_event=False,
145145
ignore_decode_errors=False,
146-
use_crc32=False,):
146+
verify_checksum=False,):
147147
"""
148148
Attributes:
149149
ctl_connection_settings: Connection settings for cluster holding
@@ -185,7 +185,7 @@ def __init__(self, connection_settings, server_id,
185185
used with 'is_mariadb'
186186
ignore_decode_errors: If true, any decode errors encountered
187187
when reading column data will be ignored.
188-
use_crc32: If true, use CRC32 4-byte for events validation, ensuring data integrity.
188+
verify_checksum: If true, verify events read from the binary log by examining checksums.
189189
"""
190190

191191
self.__connection_settings = connection_settings
@@ -208,7 +208,7 @@ def __init__(self, connection_settings, server_id,
208208
only_events, ignored_events, filter_non_implemented_events)
209209
self.__fail_on_table_metadata_unavailable = fail_on_table_metadata_unavailable
210210
self.__ignore_decode_errors = ignore_decode_errors
211-
self.__use_crc32 = use_crc32
211+
self.__verify_checksum = verify_checksum
212212

213213
# We can't filter on packet level TABLE_MAP and rotate event because
214214
# we need them for handling other operations
@@ -539,7 +539,7 @@ def fetchone(self):
539539
self.__freeze_schema,
540540
self.__fail_on_table_metadata_unavailable,
541541
self.__ignore_decode_errors,
542-
self.__use_crc32,)
542+
self.__verify_checksum,)
543543

544544
if binlog_event.event_type == ROTATE_EVENT:
545545
self.log_pos = binlog_event.event.position

Diff for: pymysqlreplication/event.py

+5-5
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ def __init__(self, from_packet, event_size, table_map, ctl_connection,
1818
freeze_schema=False,
1919
fail_on_table_metadata_unavailable=False,
2020
ignore_decode_errors=False,
21-
use_crc32=False,):
21+
verify_checksum=False,):
2222
self.packet = from_packet
2323
self.table_map = table_map
2424
self.event_type = self.packet.event_type
@@ -28,22 +28,22 @@ def __init__(self, from_packet, event_size, table_map, ctl_connection,
2828
self.mysql_version = mysql_version
2929
self._fail_on_table_metadata_unavailable = fail_on_table_metadata_unavailable
3030
self._ignore_decode_errors = ignore_decode_errors
31-
self._use_crc32 = use_crc32
31+
self._verify_checksum = verify_checksum
3232
self._is_event_valid = None
3333
# The event have been fully processed, if processed is false
3434
# the event will be skipped
3535
self._processed = True
3636
self.complete = True
37-
self._validate_event()
37+
self._verify_event()
3838

3939
def _read_table_id(self):
4040
# Table ID is 6 byte
4141
# pad little-endian number
4242
table_id = self.packet.read(6) + b"\x00\x00"
4343
return struct.unpack('<Q', table_id)[0]
4444

45-
def _validate_event(self):
46-
if not self._use_crc32:
45+
def _verify_event(self):
46+
if not self._verify_checksum:
4747
return
4848

4949
self.packet.rewind(1)

Diff for: pymysqlreplication/packet.py

+3-3
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ def __init__(self, from_packet, table_map,
106106
freeze_schema,
107107
fail_on_table_metadata_unavailable,
108108
ignore_decode_errors,
109-
use_crc32,):
109+
verify_checksum,):
110110
# -1 because we ignore the ok byte
111111
self.read_bytes = 0
112112
# Used when we want to override a value in the data buffer
@@ -136,7 +136,7 @@ def __init__(self, from_packet, table_map,
136136
if use_checksum:
137137
event_size_without_header = self.event_size - 23
138138
else:
139-
use_crc32 = False
139+
verify_checksum = False
140140
event_size_without_header = self.event_size - 19
141141

142142
self.event = None
@@ -154,7 +154,7 @@ def __init__(self, from_packet, table_map,
154154
freeze_schema=freeze_schema,
155155
fail_on_table_metadata_unavailable=fail_on_table_metadata_unavailable,
156156
ignore_decode_errors=ignore_decode_errors,
157-
use_crc32=use_crc32)
157+
verify_checksum=verify_checksum)
158158
if self.event._processed == False:
159159
self.event = None
160160

Diff for: pymysqlreplication/tests/test_basic.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -539,13 +539,13 @@ def create_binlog_packet_wrapper(pkt):
539539
self.stream._BinLogStreamReader__freeze_schema,
540540
self.stream._BinLogStreamReader__fail_on_table_metadata_unavailable,
541541
self.stream._BinLogStreamReader__ignore_decode_errors,
542-
self.stream._BinLogStreamReader__use_crc32,)
542+
self.stream._BinLogStreamReader__verify_checksum,)
543543
self.stream.close()
544544
self.stream = BinLogStreamReader(
545545
self.database,
546546
server_id=1024,
547547
blocking=False,
548-
use_crc32=True
548+
verify_checksum=True
549549
)
550550
# For event data, refer to the official document example data of mariaDB.
551551
# https://mariadb.com/kb/en/query_event/#example-with-crc32

0 commit comments

Comments
 (0)