flow-go icon indicating copy to clipboard operation
flow-go copied to clipboard

Published 20 hours ago • verified publisher icon onflow

Update error type for pruning functions

Open durkmurder opened this issue 3 years ago • 1 comments

This revealed a bit of a terminology conflict:

  • The error is using Height, while we prune here by view.

A potential solution would be to relax the name of the error a bit; e.g. we could call it AlreadyPrunedError. It is much less specific ... but maybe that makes the error more broadly applicable 🤷

Nothing we have to address in this PR, just shopping for ideas. cc @zhangchiqing

Originally posted by @AlexHentschel in https://github.com/onflow/flow-go/pull/1198#r703989274

durkmurder avatar Sep 08 '21 09:09 durkmurder