-
Notifications
You must be signed in to change notification settings - Fork 433
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
Feature/flatten serialization #2056
Conversation
If a
Originally posted by @blythed in #2049 (comment) |
0799ec1
to
bafde5b
Compare
dd9bf63
to
95cec4e
Compare
return r | ||
|
||
def delete_artifact(self, r: t.Dict): |
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.
I'm not really sure we need this method.
Since the artifacts and files are flat in Document.encode
, we can
just directly with save_bytes
, save_file
.
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.
Not a big deal however - can be refactored later.
return current | ||
|
||
|
||
def parse_query( |
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.
TODO: something with a != b
, a == b
, a >= b
, a <= b
etc..
|
||
|
||
# TODO: (New) remove the unused code | ||
def _from_dict(r: t.Any, db: None = None) -> t.Any: |
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, no longer needed.
|
||
def __repr__(self): | ||
return '<EMPTY>' | ||
|
||
|
||
class IntermidiaType: |
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.
Intermediate type?
symbol_table = self.db[identifier] | ||
symbol_table = symbol_table.relabel( | ||
# TODO: Check for folds | ||
{'output': identifier, '_fold': f'fold.{identifier}'} |
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.
Why this?
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.
Looking very good, just a few things I would like to understand with @jieguangzhou and @kartik4949.
bb690fe
to
5e54403
Compare
5e54403
to
bb690fe
Compare
- Support table-level schema - Support unified query and insert methods - Support default field queries in Ibis
145c602
to
3bcb755
Compare
3bcb755
to
ad0f535
Compare
Description
Related Issues
Checklist
make unit-testing
andmake integration-testing
successfully?Additional Notes or Comments