Skip to content

Fix a panic: RefCounter update should only be added to WAL after we enable it

Ivaylo Novakov requested to merge ivo/fix_empty_rc_update into master

MERGE REQUEST

MR Guidelines

Overview

An empty RefCounter WAL update was making its way into the transaction, causing the WAL to panic.
With this MR the update will only go into the transaction if the RefCounters are enabled (all the checks for "testing" are removed).

Example for Visual Changes

Checklist

Review and complete the checklist to ensure that the MR is complete before assigned to an approver.

  • All new methods or updated methods have clear docstrings
  • Testing added or updated for new methods
  • Any new packages are added to Makefile and .gitlab-ci.yml
  • API documentation updated for API updates
  • Module README.md updated for changes to workflow
  • Issue added to Sia-UI repo for new supporting features
  • Changelog File Created

Issues Closed

Edited by David Vorick

Merge request reports