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

feat: update index json #379

Merged
merged 6 commits into from
Jan 28, 2023
Merged

feat: update index json #379

merged 6 commits into from
Jan 28, 2023

Conversation

elrrrrrrr
Copy link
Member

@elrrrrrrr elrrrrrrr commented Jan 20, 2023

多同步源方案之后,原有 srouce_registry 配置仅初始化时消费, 更新 / 状态信息相关字段

  1. 使用 information_schema 替换 id 计算,解决部分 db id 自增不连续的问题
  2. 添加 upstream_registries 列表,返回对应 changesStreamTaskData 以及 registry 信息
  3. source_registry , changes_stream_registry, sync_changes_steam 标记为 Legacy 字段,暂不移除
  4. 新增 rawQueryUtil 处理 getCount 类型查询逻辑

@elrrrrrrr elrrrrrrr added the enhancement New feature or request label Jan 20, 2023
@codecov
Copy link

codecov bot commented Jan 20, 2023

Codecov Report

Merging #379 (d87f421) into master (0cc348d) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #379      +/-   ##
==========================================
- Coverage   97.47%   97.44%   -0.03%     
==========================================
  Files         151      152       +1     
  Lines       12918    13097     +179     
  Branches     1681     1689       +8     
==========================================
+ Hits        12592    12763     +171     
- Misses        326      334       +8     
Impacted Files Coverage Δ
app/common/adapter/binary/AbstractBinary.ts 94.00% <100.00%> (+0.06%) ⬆️
app/common/adapter/binary/ApiBinary.ts 90.90% <100.00%> (+0.58%) ⬆️
app/common/adapter/binary/BucketBinary.ts 100.00% <100.00%> (ø)
app/common/adapter/binary/CypressBinary.ts 100.00% <100.00%> (ø)
app/common/adapter/binary/ElectronBinary.ts 100.00% <100.00%> (ø)
app/common/adapter/binary/GithubBinary.ts 92.15% <100.00%> (+0.49%) ⬆️
app/common/adapter/binary/ImageminBinary.ts 100.00% <100.00%> (ø)
app/common/adapter/binary/NodeBinary.ts 100.00% <100.00%> (ø)
app/common/adapter/binary/NodePreGypBinary.ts 100.00% <100.00%> (ø)
app/common/adapter/binary/NwjsBinary.ts 100.00% <100.00%> (ø)
... and 18 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Member

@fengmk2 fengmk2 left a comment

Choose a reason for hiding this comment

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

LGTM

@elrrrrrrr elrrrrrrr changed the title feat: update index json WIP: feat: update index json Jan 21, 2023
@elrrrrrrr elrrrrrrr force-pushed the udpate-index-json branch 3 times, most recently from 8de7c92 to 8dafd33 Compare January 21, 2023 07:44
@elrrrrrrr elrrrrrrr changed the title WIP: feat: update index json feat: update index json Jan 21, 2023
@elrrrrrrr elrrrrrrr changed the title feat: update index json WIP: feat: update index json Jan 22, 2023
@elrrrrrrr elrrrrrrr changed the title WIP: feat: update index json feat: update index json Jan 28, 2023
@fengmk2 fengmk2 merged commit bce6e79 into master Jan 28, 2023
@fengmk2 fengmk2 deleted the udpate-index-json branch January 28, 2023 06:59
fengmk2 pushed a commit that referenced this pull request Jan 28, 2023
[skip ci]

## [3.2.0](v3.1.2...v3.2.0) (2023-01-28)

### Features

* update index json ([#379](#379)) ([bce6e79](bce6e79))
@github-actions
Copy link

🎉 This PR is included in version 3.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants