Technical details on how to store refs #4

Nova issue

Aberto

aberto por jcgruenhage 2018-06-10 10:35:14 +00:00 · 2 comentários

jcgruenhage comentou 2018-06-10 10:35:14 +00:00

Proprietário

Copiar link

I think the main question here is whether to prefer one larger state event over a few smaller ones. We could put all the refs into one large state event, or we could have a state event per ref with the ref being the key.

For the latter approach, we would still need to store the list of all refs somewhere, unless I missed some API to list all state keys for a given event type.

If the power level requirement was on the tuple (event_type, state_key) instead of just event_type, splitting it into multiple events would allow for setting different power level requirements per branch, but a quick read through https://matrix.org/docs/spec/client_server/r0.3.0.html#m-room-power-levels implies that this won't work.

I think the main question here is whether to prefer one larger state event over a few smaller ones. We could put all the refs into one large state event, or we could have a state event per ref with the ref being the key. For the latter approach, we would still need to store the list of all refs somewhere, unless I missed some API to list all state keys for a given event type. If the power level requirement was on the tuple (event_type, state_key) instead of just event_type, splitting it into multiple events would allow for setting different power level requirements per branch, but a quick read through https://matrix.org/docs/spec/client_server/r0.3.0.html#m-room-power-levels implies that this won't work.

jcgruenhage adicionou o rótulo

Bikeshedding

2018-06-10 10:35:26 +00:00

jcgruenhage comentou 2018-06-10 11:35:17 +00:00

Autor

Proprietário

Copiar link

So basically either we store an state event of the type git.references and an empty state key, with a content like this:

Large state event:

git.references, empty state key

{
  "tags": [
    {
      "name": "v0.1.0",
      "hash": "sha1ofsomegitobject"
    }
  ],
  "heads": [
    {
      "name": "master",
      "hash": "sha1ofsomegitobject"
    }
  ]
}

Smaller per ref state events

git.references, empty state key

{
  "tags": [
    "v0.1.0"
  ],
  "heads": [
    "master"
  ]
}

git.references.tag, state key v0.1.0

"sha1ofsomegitobject"

git.references.head, state key master

"sha1ofsomegitobject"

Performance

I don't know which of those will perform better or worse, maybe homeserver implementers have a better idea here.

So basically either we store an state event of the type `git.references` and an empty state key, with a content like this: ## Large state event: ### `git.references`, empty state key ```json { "tags": [ { "name": "v0.1.0", "hash": "sha1ofsomegitobject" } ], "heads": [ { "name": "master", "hash": "sha1ofsomegitobject" } ] } ``` ## Smaller per ref state events ### `git.references`, empty state key ```json { "tags": [ "v0.1.0" ], "heads": [ "master" ] } ``` ### `git.references.tag`, state key `v0.1.0` ```json "sha1ofsomegitobject" ``` ### `git.references.head`, state key `master` ```json "sha1ofsomegitobject" ``` ## Performance I don't know which of those will perform better or worse, maybe homeserver implementers have a better idea here.

jcgruenhage comentou 2018-07-24 17:47:39 +00:00

Autor

Proprietário

Copiar link

So I currently prefer the smaller events, most of all because that means that we can merge people pushing to different branches on different servers at the same time. If multiple people push to the same branch at the same time, we'll definitely loose all pushes except one.

So I currently prefer the smaller events, most of all because that means that we can merge people pushing to different branches on different servers at the same time. If multiple people push to the same branch at the same time, we'll definitely loose all pushes except one.

Acesse para participar desta conversação.

Nenhum branch/tag especificado

Branches Tags

master

Etiquetas

Limpar etiquetas   

Bikeshedding

Sem etiqueta

Bikeshedding

Marco

Limpar marco

Nenhum item

Sem marco

Responsáveis

Limpar responsáveis

Sem responsável

1 participante(s)

Notificações

Inscrever-se

Data limite

A data limite é inválida ou está fora do intervalo. Por favor, use o formato 'dd/mm/aaaa'.

Data limite não informada.

Dependências

Nenhuma dependência definida.

Referência: jcgruenhage/git-on-matrix#4

Nenhuma descrição fornecida.