Skip to content

add AttrDictCursor#633

Closed
caitinggui wants to merge 1 commit into
PyMySQL:masterfrom
caitinggui:master
Closed

add AttrDictCursor#633
caitinggui wants to merge 1 commit into
PyMySQL:masterfrom
caitinggui:master

Conversation

@caitinggui

@caitinggui caitinggui commented Jan 15, 2018

Copy link
Copy Markdown

Let results allow attribute-style access, and I think it can be a bit user-friendly.
data = {"row1": "the first row", "row2": "foo"}
data["row1"] VS data.row1

@methane

methane commented Jan 15, 2018

Copy link
Copy Markdown
Member

I don't like it

@methane methane closed this Jan 15, 2018
@caitinggui

Copy link
Copy Markdown
Author

But it can be more convenient and without any performance loss.

@methane

methane commented Jan 15, 2018

Copy link
Copy Markdown
Member

You can use it even if PyMySQL doesn't provide it.
And why do you think your AttrDict is world best? Is yours better than attrdict on PyPI?

Anyway, I want to focus low-level, performance and stability.
I don't want to add user-friendly parts I maintain.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants