tildeverse
/
rfcs
4
6
Fork 6
Code Issues 5 Pull Requests 1 Releases Wiki Activity

Added field to appmetadata table in RFC 3 #6

Merged
khuxkm merged 1 commits from center/rfcs:tilde-center into master 2019-03-04 20:44:07 +00:00
Conversation 1 Commits 1 Files Changed 1 +2
aewens commented 2019-02-21 16:52:04 +00:00
Contributor
Copy Link

A gross oversight on my part, but the lack of a second foreign key to another appdata entry makes queries between appdata and appmetadata horribly inefficient for its desired use case.

Previously, if a user wanted to make a service like Facebook with one appdata entry (e.g. a user) link to many other appdata entries (e.g. friends) using the appmetadata table, the appmetadata entry would need to convey it was a friend attribute and the ID of the other user in the VARCHAR(32) value field.

Now, for the same appdata-to-appdata relationship, the appmetadata could link to the user, the friend user, and use the value field to denote the relationship of these two entries as “friend”. This new field to appmetadata enabled the power behind using a relational database and can turn what would be multiple SQL queries before into just one.

A gross oversight on my part, but the lack of a second foreign key to another appdata entry makes queries between appdata and appmetadata horribly inefficient for its desired use case. Previously, if a user wanted to make a service like Facebook with one appdata entry (e.g. a user) link to many other appdata entries (e.g. friends) using the appmetadata table, the appmetadata entry would need to convey it was a friend attribute and the ID of the other user in the VARCHAR(32) value field. Now, for the same appdata-to-appdata relationship, the appmetadata could link to the user, the friend user, and use the value field to denote the relationship of these two entries as “friend”. This new field to appmetadata enabled the power behind using a relational database and can turn what would be multiple SQL queries before into just one.
aewens commented 2019-03-04 17:17:44 +00:00
Author
Contributor
Copy Link

Is there anything I need to change to have this PR accepted?

Is there anything I need to change to have this PR accepted?
khuxkm closed this pull request 2019-03-04 20:44:07 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: tildeverse/rfcs#6
No description provided.
Delete Branch "center/rfcs:tilde-center"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?