Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move TaskHeader to golem-messages #3557

Closed
jiivan opened this issue Nov 15, 2018 · 1 comment

Comments

Projects
None yet
2 participants
@jiivan
Copy link
Contributor

commented Nov 15, 2018

Also remove FixedTaskHeader and leave only TaskHeader. Since both can only be modified by requestor there is no need for both entities.

related: #80 #2407

@mfranciszkiewicz

This comment has been minimized.

Copy link
Collaborator

commented Dec 12, 2018

@jiivan @shadeofblue Task headers are supposed to be (mainly) immutable and we have to resolve this issue somehow.

What I can think of is:

  • add a creation timestamp property to the immutable part of the header (there has to be one)
  • make task_id a derivative of Hash(FixedTaskHeader) and exclude it from the fixed part
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.