aloknerurkar

Not sure if this is correct. We can have `/` in the index document. The way it works is for each folder you can configure the index document. So you...

> Please please please make sure the reserve store doesn't mess with local pinning as described in #3037 I will update you with more details. This is definitely the way!

If the stamp was invalid the chunk would actually be marked sync. This was one of issues we fixed where chunks in the push queue were being tried again and...

So I dont think we should fix this. `dev` mode is not meant to test for correctness. This should only be used to check API responses for integration on the...

If you are testing postage contract logic (topup/dilute etc), you should be using the actual contract and not a dummy implementation. I dont see any value in testing your logic...

@vladopajic So have you seen any problems with this? If this is a problem, I would like to see some pprof data to point to this. Also, we will need...

> What overhead of `RWLock` do you have in mind? > > The overlocking does not have anything to do with `RWLock` (that's additional improvement); it happens because previously this...

@vladopajic I agree with Peter that there is no need for this function. This existing function is compact enough and just moving the unlock below the if loop would be...

> @aloknerurkar > > I can't comprehend why would you decide to block change because it encapsulated functionality in individual function. > > Certainly we have deadlock situation here so...

@istae Is there an issue for this? I dont think this is the right usage of TopUp. TopUp is the topup amount per chunk. This is not the new value...