Skip to content
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

RecordSet instead of Table #348

Closed
hrennau opened this issue Sep 28, 2019 · 3 comments
Closed

RecordSet instead of Table #348

hrennau opened this issue Sep 28, 2019 · 3 comments
Labels
lang Relates to the Ballerina language specification status/idea An idea that is not yet a worked-out proposal

Comments

@hrennau
Copy link

hrennau commented Sep 28, 2019

Description:
The [Preview] Tables type perhaps introduces unnecessary heterogeneity? Proposal: consider replacing type Tables with a new type RecordSet. A recordset-type-descriptor identifies a record type, plus a possibly empty list of field names used as primary keys. Benefits: reuse of syntactical constructs, reuse of the existing concept "field" (getting rid of columns), gratuitous introduction of default values (not yet included in the Table model).

Also to be considered: shifting the concept of "primary key" into Record, at the same time evolving it into the more abstract notion of "identity defining fields". So it would be conceivable to extend the definition of Record to include optional flags marking fields as contributing to semantic identity, something which might be valuable beyond the use of databases. Primary keys would then simply be an application of semantic identity.

Suggested Labels:
Tables, Records, RecordSets, Semantic Identity

@jclark
Copy link
Collaborator

jclark commented Oct 3, 2019

Thank you for your comment. There is a proposal for tables, which is fills out what is in the spec.

https://github.com/ballerina-platform/ballerina-spec/blob/master/lang/proposals/table/table.md

I am trying to gather discussion on that in issue #354. Could you make your comment there? Or I can copy it. We have not made any decision about whether to adopt the proposal, and are open to alternative suggestions.

@jclark
Copy link
Collaborator

jclark commented Oct 6, 2019

I have mentioned this in #354 (comment)

@jclark jclark added status/idea An idea that is not yet a worked-out proposal lang Relates to the Ballerina language specification labels Oct 6, 2019
@jclark
Copy link
Collaborator

jclark commented Mar 8, 2020

There is a new proposal for tables, which is more like a RecordSet.

@jclark jclark closed this as completed Mar 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang Relates to the Ballerina language specification status/idea An idea that is not yet a worked-out proposal
Projects
None yet
Development

No branches or pull requests

2 participants