Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Implement Collator #11

Merged
merged 10 commits into from
Oct 8, 2019
Merged

Implement Collator #11

merged 10 commits into from
Oct 8, 2019

Conversation

bkchr
Copy link
Member

@bkchr bkchr commented Jun 18, 2019

(Just concentrate on the collator crate)

Open questions:

  • What do we do with max_duration of propose?
  • Do we want to return more detailed errors in produce_candidate?

@bkchr bkchr requested a review from rphmeier June 18, 2019 14:27
inherent_data,
Default::default(),
//TODO: Fix this.
Duration::from_secs(6),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure what to do here, but it is worth noting that polkadot-collator times out collation futures that take longer than a given duration. Maybe we can pass that timeout along to ProduceCandidate somehow.

native_executor_instance!(
pub LocalExecutor,
runtime::api::dispatch,
runtime::native_version,
include_bytes!("../../runtime/wasm/target/wasm32-unknown-unknown/release/cumulus_test_runtime.compact.wasm")
runtime::WASM_BINARY
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
runtime::WASM_BINARY
runtime::WASM_BINARY,

@bkchr bkchr merged commit c10f05e into master Oct 8, 2019
@bkchr bkchr deleted the bkchr-collator branch October 8, 2019 07:53
imstar15 pushed a commit to imstar15/cumulus that referenced this pull request Nov 16, 2022
Maharacha pushed a commit to Maharacha/cumulus that referenced this pull request May 10, 2023
yrong pushed a commit to yrong/cumulus that referenced this pull request May 13, 2023
* Add support for new pallets

* Update RPC handler for OutboundQueue
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants