Skip to content
This repository was archived by the owner on Oct 25, 2024. It is now read-only.
This repository was archived by the owner on Oct 25, 2024. It is now read-only.

Does this check in capella-1.11.5 branch break bundle cancellations? #52

Open

Description

@bkellerman

A block with cancelled bundles could have less profit.

if payload.full == nil || fees.Cmp(payload.fullFees) > 0 {

Activity

  1. changed the title [-]Does this check break bundle cancellations?[/-] [+]Does this check in `capella-1.11.5` branch break bundle cancellations?[/+] on Mar 22, 2023
  2. Ruteri commented on Mar 22, 2023

    @Ruteri
    Collaborator

    I think it's alright
    The check here makes it so that only the most valuable payload makes it into the relay submission code. But all of those payloads have the cancelled bundles in consideration, and even if the resulting payload has a smaller profit than the previous relay submission it should be still sent.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions