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

Annim 元数据仓库 #77

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Annim 元数据仓库 #77

wants to merge 7 commits into from

Conversation

SeraphJACK
Copy link
Member

No description provided.

@SeraphJACK SeraphJACK requested a review from a team April 14, 2023 00:54
src/06.anniv/09.search.md Outdated Show resolved Hide resolved
src/06.anniv/09.search.md Outdated Show resolved Hide resolved

#### 返回

返回包含所有 `Tag` 的数组,类型为 `TagInfo[]` 。
Copy link
Member

Choose a reason for hiding this comment

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

元数据相关的类型定义可以从 Anniv 里移出来了

Copy link
Member Author

Choose a reason for hiding this comment

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

Anniv 的分享还是会用到导出格式的

Copy link
Member

Choose a reason for hiding this comment

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

指放在 annim 里,anniv 也引这边的类型

"last_update": 1639631487
"last_update": 1639631487,
// 用于提供元数据的 `Annim` 服务器 URL
"meta_server": "https://annim.example.com"
Copy link
Member

Choose a reason for hiding this comment

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

这部分可以参考一下另一个pr,我找一下

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

annim 应该是 required ,而 repo 和预构建 db 都应该用 annim 提供(?

Copy link
Member

@Yesterday17 Yesterday17 Apr 14, 2023

Choose a reason for hiding this comment

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

No, Annim 是 repo 的延伸,这样 repo 和 Annim 就是一对多的。Annim 和 db 一个是动态的数据源,一个是静态的数据源,也应该是平级的。

Copy link
Member Author

Choose a reason for hiding this comment

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

annim 某种意义上来说是对 repo 的封装,为什么是平级的。而且要让客户端处理 git 也很麻烦

Copy link
Member

Choose a reason for hiding this comment

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

哦,上面写错了, annim 和 db 是平级的

Copy link
Member

Choose a reason for hiding this comment

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

需要解决的其实是:

  1. 不管最终 repo 以什么形态出现,我们肯定会保留一种备份并公开元数据仓库的手段。
  2. 从这份公开的元数据备份中 Host 一个 Annim 并添加到客户端之后,我们需要让支持的所有 Annil 都自动选用这个 Annim,而不是每个用了这个 repo 的服务器都要单独配置一遍。

Copy link
Member Author

Choose a reason for hiding this comment

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

那问题就是现在元数据的来源是抽象的,没有一个可用的 identifier 。

Copy link
Member

@Yesterday17 Yesterday17 Apr 18, 2023

Choose a reason for hiding this comment

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

那我们给 repo 加一个 uuid?然后给版本加一个 hash

Copy link
Contributor

Choose a reason for hiding this comment

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

那我们给 repo 加一个 uuid?然后给版本加一个 hash

参考 nix flake 的 input?

src/08.annim/01.protocol.md Outdated Show resolved Hide resolved
src/08.annim/01.protocol.md Outdated Show resolved Hide resolved
src/08.annim/01.protocol.md Show resolved Hide resolved
src/08.annim/01.protocol.md Outdated Show resolved Hide resolved
src/08.annim/01.protocol.md Outdated Show resolved Hide resolved
src/08.annim/01.protocol.md Outdated Show resolved Hide resolved
@SeraphJACK SeraphJACK requested a review from Yesterday17 April 21, 2023 05:13
src/08.annim/01.protocol.md Outdated Show resolved Hide resolved
src/08.annim/01.protocol.md Show resolved Hide resolved
src/08.annim/01.protocol.md Show resolved Hide resolved
src/08.annim/01.protocol.md Outdated Show resolved Hide resolved
src/08.annim/01.protocol.md Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

4 participants