firefly icon indicating copy to clipboard operation
firefly copied to clipboard

Run performance testing for 1.1 release

Open awrichar opened this issue 2 years ago • 1 comments

Child of https://github.com/hyperledger/firefly/issues/563

Need to test the latest code prior to doing a 1.1 release and check for bugs/performance bottlenecks. Should test with both firefly-ethconnect and with the new firefly-evmconnect.

awrichar avatar Aug 19 '22 14:08 awrichar

#563 updated with the results of the latest (~4 day) performance test run. No major issues at this time - will test at least once more once a release candidate is selected.

awrichar avatar Aug 22 '22 18:08 awrichar

Link to the detailed Discord thread: https://discord.com/channels/905194001349627914/1004496575059476570/1020402585628319845

peterbroadhurst avatar Nov 21 '22 18:11 peterbroadhurst

The summary observations from the final run for 1.1 were:

  • We are now spending a lot of time processing receipts, and updating operations, with how chatty EVMConnect is. Large payloads, more often. Optimizing this would be best achieved by switching the webhooks from EVMconnect to just be status changes, and making the large payload be queried through on demand (as we made changes like this on listeners for example).
  • The batch assembly loop is running hot too. Uploading blobs is the highest user, followed by flushNonceState.
  • The overhead of the eventLoop is also high. We've discussed this many times, and I'm not going to add more here.
  • Verbose debug logging is a non-trivial amount of the overall usage at this point. Think we can close the 1.1 activities here. No outstanding bugs. Room for optimization as the community decides going forwards, based on specific use cases and functions in projects.

peterbroadhurst avatar Nov 21 '22 18:11 peterbroadhurst

Suggest closing this as FF 1.2 has just been released

matthew1001 avatar Feb 08 '23 16:02 matthew1001