-
Notifications
You must be signed in to change notification settings - Fork 55
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
Revised design for table type #47
Comments
A few more issues:
|
We will also need to
|
See issue #286 for a proposal that covers these. |
It is clear that the approach of the proposal in #286 is not the right approach. We need an approach that does not clone tables. This will need a fresh proposal. In the meantime, we have a query feature that is not table specific (see #398, #340), and we have streams #406, which cover the iterator-type usage of tables in the existing implementation. We are no longer trying to make a table be a transparent proxy for an SQL table. This is inconsistent with Ballerina's approach to network error handling. |
There is now a new proposal. |
We can split the changes into the following:
|
Should be `table<map<anydata>>`. Part of #47.
Part of #47. Rewrite section on table-type-descriptor. Rewrite section on table-constructor.
User control over duplicate keys in a query-expr that creates a table is a separate issue #481. |
To handle new table design. Part of #47.
Made the iterator problem be a separate issue #485. |
There's a discussion here:
https://groups.google.com/forum/#!msg/ballerina-dev/b4GM_sGXA64/GJV75EaCGQAJ
The text was updated successfully, but these errors were encountered: