Copilot commented on code in PR #1300:
URL:
https://github.com/apache/cassandra-python-driver/pull/1300#discussion_r3584233415
##########
cassandra/policies.py:
##########
@@ -22,17 +22,16 @@
from threading import Lock
import socket
import warnings
+from cassandra import WriteType as WT
+from cassandra import ConsistencyLevel, OperationTimedOut
Review Comment:
PEP 8 import grouping: separate standard-library imports from local
`cassandra` imports with a blank line (and consider consolidating the two `from
cassandra import ...` lines).
##########
cassandra/cqltypes.py:
##########
@@ -1006,9 +1020,11 @@ def evict_udt_class(cls, keyspace, udt_name):
@classmethod
def apply_parameters(cls, subtypes, names):
- keyspace = subtypes[0].cass_parameterized_type() # when parsed from
cassandra type, the keyspace is created as an unrecognized cass type; This gets
the name back
+ # when parsed from cassandra type, the keyspace is created as an
unrecognized cass type; This resolves the name
Review Comment:
Minor comment grammar/style: start the sentence with a capital letter and
avoid capitalizing the second clause after the semicolon.
##########
cassandra/util.py:
##########
@@ -30,16 +30,14 @@
import sys
import time
import uuid
+from cassandra import DriverException
Review Comment:
PEP 8 import grouping: add a blank line between standard-library imports and
local `cassandra` imports for consistency with other modules.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
