Consider making offers expire
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 20/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Necesita aclaración
- Estado de actividad
- Estancado
Línea de trabajo
Comienza leyendo CAP0-0003 y la discusión del issue sobre el comportamiento del order book. Compara los enfoques propuestos de purge y deterministic-expiration y determina después si el protocolo necesita una semántica y una especificación definidas; actualmente, el issue no indica archivos de implementación, pruebas ni criterios concretos de finalización.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
One of the things we learned with the protocol 10 rollout with changes like CAP0-0003 is that it's becoming increasingly hard to not break certain client business logic that rely on undocumented (something we can improve on for sure) or undefined behavior.
One aspect in particular is on how the order book functions: while the order book can "at any time" cancel or modify outstanding order, the issue is that those events happen so rarely that people may not realize that this is the case.
A simple way to turn the "improbable" into "certainty" (therefore hardening client code), would be to simply move semantics that are closer to how other markets work: force long lasting orders to be recreated at a regular time.
A couple possible approaches to start the discussion.
The purge
There could be a recurrent "purge" of the order book that deletes X% of the orders every T ledgers (both of these would be network parameters).
This approach seems to be close to what we want: non deterministic (due to the random nature) deletion of orders, which forces clients to not rely on any particular behavior of how the order book state is managed.
By having many ledgers between purge events, the "pseudo randomness" that comes with simple things like "ledger hash" is quite strong.
Deterministic expiration
We could also add an expiration time as a network parameter, we would pick something like a week or two as a starting point and offers would just automatically get deleted when they reach their expected life time.
The issue with that approach is that it only partially solves the problem: what if we want in a future upgrade or a future version of the protocol to delete certain offers? If client code relies on certain timing, it won't work.
A nice thing about this solution is that it's basically how standard order books work (be it "good for day" or "good until cancelled" that really means 30 or 60 days for most brokerage firms).
- Lenguaje dominante
- RPC
- Estrellas
- 632
- Forks
- 399
- Merge medio
- 1 d 23 h
- PR fusionados (30 d)
- 6
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de stellar/stellar-protocol
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
stellar/stellar-protocol#2010 ·
-
SEP-43: contract account support Abiertostale
Dificultad 5/5 Más de una semana Aptitud para principiantes 42/100
stellar/stellar-protocol#1928 · 6 comentarios ·
-
needs draft SEP
stellar/stellar-protocol#773 · 12 comentarios · 1 asignado ·
-
Merging account with trustlines AbiertoCAP help wanted needs draft
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
stellar/stellar-protocol#324 · 11 comentarios ·
-
help wanted needs draft SEP
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
stellar/stellar-protocol#316 · 4 comentarios ·
Todos los issues de stellar/stellar-protocol
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
bug examples invalid iris
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
ai_reviewed
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
ydb-platform/ydb#53838 · 3 comentarios ·