Fix bugs in block polling logic and serialization

[iuwqyir]

TL;DR

Improved error handling and logging in the poller and serializer components.

What changed?

• Enhanced logging in the poller to include the number of blocks being polled.
• Added a check to skip polling when the start block is greater than the latest block.
• Fixed a miscalculation issue in block range calculation due to pointer references.
• Improved error handling in the serializer for nil block results.
• Modified the serializer to use a dynamic slice instead of a fixed-size array for results.

How to test?

1. Run the poller with various block ranges, including edge cases where the start block is greater than the latest block.
2. Simulate scenarios where the RPC returns nil block results and verify proper error handling.
3. Test the serializer with a large number of blocks to ensure dynamic slice allocation works correctly.
4. Verify that the new log messages appear correctly in the debug output.

Why make this change?

These changes improve the robustness and reliability of the polling and serialization processes. By adding more detailed logging and error handling, it becomes easier to diagnose issues and prevent potential crashes. The modifications also optimize memory usage by using dynamic slices and fix a potential integer overflow bug in block range calculations.

[iuwqyir]

• Fix poller creating gaps if pollUntilBlock is configured #85
• Fix bugs in block polling logic and serialization #84 👈
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @iuwqyir and the rest of your teammates on Graphite

[iuwqyir]

Merge activity

• Oct 2, 10:26 AM EDT: @iuwqyir started a stack merge that includes this pull request via Graphite.
• Oct 2, 10:26 AM EDT: @iuwqyir merged this pull request with Graphite.