-
-
Notifications
You must be signed in to change notification settings - Fork 98
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
Migrate from "id" to "guid" #350
Draft
johnjohndoe
wants to merge
3
commits into
master
Choose a base branch
from
id-guid-minimal-migration
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
466fcbc
to
339568d
Compare
cketti
previously approved these changes
Dec 24, 2020
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.
LGTM 👍
|
||
private static final String SESSIONS_TABLE_CREATE = | ||
"CREATE TABLE " + SessionsTable.NAME + " (" + | ||
BaseColumns._ID + " INTEGER PRIMARY KEY AUTOINCREMENT," + | ||
Columns.SESSION_ID + " TEXT, " + |
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.
We should use this opportunity to remove the SESSION_ID
column from the database.
74e46d4
to
1749388
Compare
1749388
to
c0cc3ed
Compare
c0cc3ed
to
0a9e184
Compare
0a9e184
to
8580291
Compare
8580291
to
dda8ebb
Compare
4fce673
to
c5e3e02
Compare
c5e3e02
to
9105cbf
Compare
9105cbf
to
7901539
Compare
7901539
to
e4d3209
Compare
e4d3209
to
14fbd5a
Compare
8a257c6
to
d64ff82
Compare
d64ff82
to
e2bc8ea
Compare
e2bc8ea
to
7d4520a
Compare
7d4520a
to
f4b8761
Compare
f4b8761
to
cb343a8
Compare
cb343a8
to
368ab4e
Compare
368ab4e
to
6397fd8
Compare
6397fd8
to
91c7213
Compare
91c7213
to
80cf3bb
Compare
8c78fb6
to
14ab9b1
Compare
14ab9b1
to
9e75f73
Compare
7717fc3
to
ae992cc
Compare
ae992cc
to
37c2693
Compare
37c2693
to
7a6a7f3
Compare
7a6a7f3
to
802ae66
Compare
71350ab
to
bc71a30
Compare
bc71a30
to
1ffa9a0
Compare
1ffa9a0
to
1248014
Compare
!!! ATTENTION !!! This change is NOT compatible to UPDATE exiting apps! + As of this commit the "id" XML attribute is no longer used as the unique token because its value cannot be guaranteed to be unique by the provider of the schedule XML. See voc/schedule#63. + Instead the "guid" XML attribute is used which guarantees unique values. + To keep this migration to a minimum only the value of the Session#sessionId field is changed. The value of the new primary key (auto-incrementing, unique through the "guid" column) is written into the "sessionId" field when a Session is queried from the database. See SessionsDatabaseRepository. + By this the value of the "id" XML attribute becomes unused hence the deprecation of the SESSION_ID database column.
!!! ATTENTION !!! This change is NOT compatible to UPDATE exiting apps! + This reset is possible and needed because the former commit is incompatible with current installations. + Improve a few table and column names.
+ Non-unique session will be discarded without telling the user.
1248014
to
04d5466
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Architecture
Feature request
A request for a new feature.
Notifications
System notifications for session alarms or schedule updates
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This branch contains the following changes:
id
toguid
.guid
values.Test scenarios
The following use cases have been identified as important to test with regards to the changes made in this pull request.
Testing
Resolves #319