search

sofastack / sofa-registry

SOFARegistry is a production-level, low-latency, high-availability service registry powered by Ant Financial.
https://www.sofastack.tech/sofa-registry/docs/Home
Apache License 2.0
653 stars 247 forks source link

chore: migrate revision servie_param field to mediumtext #257

Closed dzdx closed 2 years ago

dzdx commented 2 years ago

Motivation:

Explain the context, and why you're making that change. To make others understand what is the problem you're trying to solve.

Modification:

大型应用发布的接口数可能会有600+， 会超过64KB限制

Result:

Fixes #.

If there is no issue then describe the changes introduced by this PR.

codecov[bot] commented 2 years ago

Codecov Report

Merging #257 (42c956b) into master (51c2f74) will increase coverage by 0.00%. The diff coverage is 80.39%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #257   +/-   ##
=========================================
  Coverage     76.60%   76.61%           
  Complexity       48       48           
=========================================
  Files           439      440    +1     
  Lines         15475    15524   +49     
  Branches       1505     1510    +5     
=========================================
+ Hits          11855    11894   +39     
- Misses         2961     2965    +4     
- Partials        659      665    +6
Impacted Files Coverage Δ
...ion/providedata/AppRevisionWriteSwitchService.java 66.66% <66.66%> (ø)
...try/jdbc/convertor/AppRevisionDomainConvertor.java 73.77% <88.00%> (+7.98%) :arrow_up:
...egistry/common/model/constants/ValueConstants.java 93.33% <100.00%> (+0.31%) :arrow_up:
...y/sofa/registry/jdbc/domain/AppRevisionDomain.java 100.00% <100.00%> (ø)
...stry/server/session/store/AbstractDataManager.java 77.14% <0.00%> (-1.91%) :arrow_down:
...ta/lease/impl/DefaultCrossDcMetaServerManager.java 84.61% <0.00%> (-1.54%) :arrow_down:
...ofa/registry/server/session/store/DataIndexer.java 90.32% <0.00%> (+4.83%) :arrow_up:

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 51c2f74...42c956b. Read the comment docs.