You don't need to introduce the constant. You've already got disableFlag
. So, feel free to use it in cmd.Flags().GetBool(disable)
:)
Blobovniczas
initialization won't return an explicit error
I think the problem is that
b.iterateExistingDBPaths(egCtx, func(p string) (bool, error)
uses errgroup's context. So when errgroup fails with an error,egCtx
is canceled, but `iterateExisting…
Blobovniczas
initialization won't return an explicit error
I believe
eg.Wait()
should return exactly the first error it has got, no?
We'll check soon
I believe that was incorrect from the very beginning. TBH, I don't remember the task context in details. But glancing at mergeOperationStreams
I conclude it has returned MaxUint64
, probably,…
Blobovniczas
initialization won't return an explicit error
If context is done, then we may get newHeight = math.MaxUint64 + 1 -> newHeight = 0
because applyOperationStream
may not return an error
My initial concern was about whether currentTagKey
overrides currentTagKeyFromSomeOtherPackageWithTheSameIOTA
. I checked it on my own: it's fine, context keeps two keys. So, nevermind!