[MCOL-4850] Refactor CMAPI to meet PEP-8 styleguide Created: 2021-08-30  Updated: 2023-12-15

Status: Stalled
Project: MariaDB ColumnStore
Component/s: cmapi
Affects Version/s: cmapi-1.3, cmapi-1.4, cmapi-1.5, cmapi-1.6
Fix Version/s: 23.10

Type: Task Priority: Minor
Reporter: Alan Mologorsky Assignee: Alan Mologorsky
Resolution: Unresolved Votes: 0
Labels: None

Issue Links:
PartOf
includes MCOL-4898 cmapi not respect custom mariadb port... Closed
includes MCOL-4939 Add a method to disable failover faci... Closed
includes MCOL-4954 Add support for new DBRM socket bytes... Closed
Relates
relates to MCOL-4851 Fix CMAPI tests and related things. Closed

 Description   
  • fix constant names to uppercase
  • fix indentations
  • fix multiline strings
  • fix docstrings to meet one format where it's possible and needed (optional)
  • fix imports (removed unused, and fix importing order)
  • fix empty strings to separate code objects

 Comments   
Comment by Alan Mologorsky [ 2021-12-16 ]

Part of styleguide fixes was made inside this tasks:

Comment by Arne Schwarck [ 2022-06-15 ]

Will this also fix the issue of sending the multiline command
cursor.execute("""DELIMITER //
select 1//
DELIMITER ;""").fetchall()
?

Comment by Alan Mologorsky [ 2022-06-17 ]

arneschwarck Sorry, looks like this is not related issue.
I'll check details about this issue.

Comment by JiraAutomate [ 2023-12-15 ]

Automated message:
----------------------------
Since this issue has not been updated since 6 weeks, it's time to move it back to Stalled.

Generated at Thu Feb 08 02:53:29 UTC 2024 using Jira 8.20.16#820016-sha1:9d11dbea5f4be3d4cc21f03a88dd11d8c8687422.