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

extend DataChange_CreatedBy, DataChange_LastModifiedBy from 32 to 64 #3552

Merged
merged 4 commits into from
Mar 13, 2021

Conversation

vdiskg
Copy link
Contributor

@vdiskg vdiskg commented Feb 19, 2021

extend DataChange_CreatedBy, DataChange_LastModifiedBy from 32 to 64

  1. make the limit of DataChange_CreatedBy, DataChange_LastModifiedBy equals to ApolloPortalDB.Users.Username
  2. oidc subject (oidc user's identification) usually have more than 32 character

@codecov-io
Copy link

Codecov Report

Merging #3552 (b742db1) into master (96c8d07) will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #3552      +/-   ##
============================================
+ Coverage     50.93%   50.94%   +0.01%     
- Complexity     2323     2324       +1     
============================================
  Files           447      447              
  Lines         13989    13989              
  Branches       1423     1423              
============================================
+ Hits           7125     7127       +2     
+ Misses         6375     6374       -1     
+ Partials        489      488       -1     
Impacted Files Coverage Δ Complexity Δ
...nfigservice/service/AccessKeyServiceWithCache.java 84.69% <0.00%> (+2.04%) 22.00% <0.00%> (+1.00%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3f3c4ed...b742db1. Read the comment docs.

@@ -7,3 +7,55 @@ alter table `GrayReleaseRule` change AppId AppId varchar(64) NOT NULL DEFAULT '
alter table `Instance` change AppId AppId varchar(64) NOT NULL DEFAULT 'default' COMMENT 'app id';
alter table `InstanceConfig` change ConfigAppId ConfigAppId varchar(64) NOT NULL DEFAULT 'default' COMMENT 'Config App Id';
alter table `ReleaseHistory` change AppId AppId varchar(64) NOT NULL DEFAULT 'default' COMMENT 'app id';

ALTER TABLE `App`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as apollo 1.8.0 was released, please put it into another folder as v180-v190

@@ -3,3 +3,47 @@
Use ApolloPortalDB;

alter table `AppNamespace` change AppId AppId varchar(64) NOT NULL DEFAULT 'default' COMMENT 'app id';

ALTER TABLE `App`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as apollo 1.8.0 was released, please put it into another folder as v180-v190

Copy link
Member

@nobodyiam nobodyiam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nobodyiam nobodyiam merged commit d6e0b01 into apolloconfig:master Mar 13, 2021
@nobodyiam nobodyiam added this to the 1.9.0 milestone Mar 16, 2021
@vdiskg vdiskg deleted the extend-username branch April 23, 2021 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants