-
Notifications
You must be signed in to change notification settings - Fork 15
[feat] Add examples and stub files for Python bindings #10
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
Conversation
1a42e61 to
90c14a4
Compare
luoyuxia
left a comment
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.
@naivedogger Thanks for the pr. Left some comments. PTAL
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 put it to bindings/python/fluss?
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.
fixed
| class FlussTable: | ||
| async def new_append_writer(self) -> AppendWriter: ... | ||
| def new_log_scanner(self) -> LogScanner: ... | ||
| def new_log_scanner_sync(self) -> LogScanner: ... |
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.
remove 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.
fixed
| class TablePath: | ||
| def __init__(self, database: str, table: str) -> None: ... | ||
| def database(self) -> str: ... | ||
| def table(self) -> str: ... |
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.
also add __hash__ and __eq__
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.
fixed
|
update |
|
@luoyuxia, thanks for your kind code review. I have addressed the comments and revised the stub file. PTAL. |
luoyuxia
left a comment
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.
+1
1092f2f to
1cf0a34
Compare
Purpose
Linked issue: close #16
Brief change log
Tests
API and Format
Documentation