elixir-omg

elixir-omg copied to clipboard

From @InoMurko's message: [the way it's now ] it would work, but it isn’t optimal! I like to think about systems with this in mind: https://medium.com/@jlouis666/stacking-theory-for-systems-design-2450e6300689 what does that mean...

pnowosie

A common scenario now is that: 1. Alice places a deposit 2. Alice's Watcher waits some finality and reports the deposit as a spendable UTXO 3. Alice wants to spend...

pdobacz

Childchain main supervisor stopped permanently after receiving unexpected result during block submission

2

```elixir 2020-03-15 21:47:02.742 [error] module=gen_server function=error_info/7 ⋅GenServer OMG.ChildChain.BlockQueue.Server terminating "** (MatchError) no match of right hand side value: {:error, %{""code"" => -32000, ""message"" => ""execution aborted (timeout = 5s)""}}" (omg_child_chain)...

unnawut

Error submitting transaction to childchain under high load

2

Scenario: Stress test of many concurrent calls to childchain/transaction.submit Blocks had large numbers of transactions: `Transaction submitted successfully {3044000, 53587}` Client started receiving `server:internal_server_error` errors. Datadog logs here: https://app.datadoghq.com/logs?cols=core_host%2Ccore_service&event&from_ts=1583403956200&index=main&live=false&messageDisplay=inline&query=host%3A%2Astress%2A+service%3Achild_chain&saved_view=80142&stream_sort=desc&to_ts=1583403956250 and...

kevsul

#1244 introduces a long running test in `State.Core` tests. ($REPO_ROOT/apps/omg/test/omg/state/core_test.exs, test `"long running full block test"`) 1. __~Check whether~ performance is __NOT__ related to the changes in the PR #1244__...

pnowosie

Enable transaction count by address.

1

Open question whether we implement this on an existing endpoint (`transaction.all`) or a new one. It would mainly be to enable the block explorer use case of showing transactions for...

kalouo

We should groom our logging outputs and place each outputs into its distinct logging levels.

achiurizo

https://github.com/omgnetwork/elixir-omg/blob/1feed637da2e5caf49b7fe2bb3a2872818baee46/config/config.exs#L164 It seems redundant because they all follow the :depositor process https://github.com/omgnetwork/elixir-omg/blob/1feed637da2e5caf49b7fe2bb3a2872818baee46/apps/omg_watcher/lib/omg_watcher/coordinator_setup.ex#L48-L62 :depositor margin is 10, exit processor finality margin is 12 - what does it really give us? Also,...

InoMurko

https://github.com/omisego/elixir-omg/blob/ae51c9814dfe516248f92c6dd2d44c2db3d2bbdf/apps/omg/lib/omg/state/transaction/payment.ex#L130-L149 rewrite this without rescue because it's hidding actual errors

InoMurko

Concern raised by @pgebal in the review of #1496. Watcher-Info consumes events from Watcher via OMG.Bus. There are several consumers to handle specific events, but they are uncoordinated. It might...

pnowosie