Replies: 4 comments 5 replies
-
|
I was planning on posting the same thing today you beat me to it. In the trace a ejected service is brought back after receiving a transfer which I don't think is correct. 1768945074 might have the same issue as well. |
Beta Was this translation helpful? Give feedback.
-
|
Also observed this issue on polkajam for both 1768945074 and 1768864701 relevant GP bit: the services in m (which are services ejected by any other service) are removed from d'
|
Beta Was this translation helpful? Give feedback.
-
|
There was a known bug in Polkajam that was discussed elsewhere some time ago. These traces are therefore likely obsolete and may need to be retired. I will try re-executing them to confirm. |
Beta Was this translation helpful? Give feedback.
-
|
@boymaas sorry but I can't find this trace in my repo. Where you found it? |
Beta Was this translation helpful? Give feedback.

Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
This trace suggest the following:
Pre-state: Service 2947170938 exists with balance 304062
Post-state: Service 2947170938 exists with balance 355082 (+51020 from transfer)
The service should survive and receive the transfer
We had an earlier discussion where I believe we agreed upon deleting the service and essentially burning the funds: #148. This is what I see happening in the trace:
Batch 1: Starting parallelized accumulation for 2 work reports, 0 pending transfers
Batch 2: Starting parallelized accumulation for 1 work reports, 1 pending transfers
So the transfer is NOT deferred to a future block - it's processed in batch 2 of the same block. Service 2947170938 accumulates in batch 2 with the incoming transfer.
The sequence in batch 2:
Both happen in parallel in the same batch. The merge then decides the outcome, which I believe should be Service 2947170938 deleted and effectively the transfer amount would be burned. I cannot find anywhere in the graypaper defined that the service needs to be restored.
See: https://github.com/w3f/jam-conformance/tree/javajam_m1/fuzz-reports/0.7.2/traces/1768864701
Beta Was this translation helpful? Give feedback.
All reactions