-
Notifications
You must be signed in to change notification settings - Fork 389
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Complete Data
to data
rename
#1431
Conversation
Renames the `InteractiveSymbol` class to `_Data`, and deprecates `InteractiveSymbol`. In version 0.11, `InteractiveSymbol` will be removed, and the `_Data` class will be renamed `Data`. This is all to be consistent with other function / class naming schemes, that have a lowercase entry point that creates an instance of the CamelCased class.
`data()` will dispatch to `odo.resource()` when needed. This change ensures that many of the edge cases in `data()` are tested.
@@ -41,19 +41,27 @@ | |||
not_an_iterator.append(pymongo.database.Database) | |||
|
|||
|
|||
class InteractiveSymbol(Symbol): | |||
"""Interactive data. | |||
class _Data(Symbol): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we just call this Data
to match the other expression/smart constructor pairs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, that's exactly the ultimate intent, and in version 0.11 we'll remove InteractiveSymbol
and rename _Data
to Data
. I made note of this in the whatsnew document.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, I should read the whole change before commenting; sorry
Complete `Data` to `data` rename
This builds on PR #1421:
Data
are replaced withdata
;Data
are deprecated;InteractiveSymbol
is deprecated and replaced by_Data
;data
create an instance of_Data
;odo.resource
are replaced byblaze.data
throughout the codebase and testsuite.In version 0.11, the following changes will take place, to complete the transition:
InteractiveSymbol
will be removed;_Data
class will be renamedData
, and calls todata
will create aData
instance;join
andJoin
,head
andHead
, etc.