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.

Sign up for GitHub

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

Jump to bottom

#999: Use scalar.Sticky for caching request #1076

Merged
merged 1 commit into from
Oct 16, 2020
Merged

#999: Use scalar.Sticky for caching request #1076

merged 1 commit into from
Oct 16, 2020

Conversation

andreoss
Copy link
Contributor

Per #999:

  • Make RqOnce use Sticky

@fabriciofx
Copy link
Contributor

@0crat refuse

@0crat
Copy link
Collaborator

0crat commented Sep 17, 2020

@0crat refuse (here)

@fabriciofx The user @fabriciofx/z resigned from #1076, please stop working. Reason for job resignation: Order was cancelled

@0crat
Copy link
Collaborator

0crat commented Sep 17, 2020

@0crat refuse (here)

@fabriciofx Job refused in 9 hours - no penalty, see §6

paulodamaso
paulodamaso suggested changes Sep 19, 2020
View reviewed changes
Copy link
Contributor

@paulodamaso paulodamaso left a comment

Choose a reason for hiding this comment

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

@andreoss Just one comment here, please take a look

src/main/java/org/takes/rq/RqOnce.java Show resolved Hide resolved
@andreoss
Copy link
Contributor Author

andreoss commented Oct 5, 2020

@paulodamaso Please take a look one more time.

@paulodamaso
Copy link
Contributor

@rultor merge

@rultor
Copy link
Collaborator

rultor commented Oct 16, 2020

@rultor merge

@paulodamaso OK, I'll try to merge now. You can check the progress of the merge here

@rultor rultor merged commit b4b7adf into yegor256:master Oct 16, 2020
@rultor
Copy link
Collaborator

rultor commented Oct 16, 2020

@rultor merge

@paulodamaso Done! FYI, the full log is here (took me 12min)

@0crat
Copy link
Collaborator

0crat commented Oct 16, 2020

Job gh:yegor256/takes#1076 is not assigned, can't get performer

@0crat 0crat removed the 0crat/scope label Oct 16, 2020
@andreoss andreoss mentioned this pull request Oct 23, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@paulodamaso paulodamaso Awaiting requested review from paulodamaso

Assignees
No one assigned
Labels
0crat/role/REV
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@andreoss @fabriciofx @0crat @paulodamaso @rultor