Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • 14:07
    niemyjski commented #472
  • 07:50
    alvin-su opened #472
  • 01:35
    niemyjski synchronize #452
  • 01:35

    niemyjski on elastic7

    Only put consumes attribute on … (compare)

  • Dec 14 16:39
    ejsmith synchronize #452
  • Dec 14 16:39

    ejsmith on elastic7

    Log the correct task id and cat… (compare)

  • Dec 14 06:25
    ejsmith synchronize #452
  • Dec 14 06:25

    ejsmith on elastic7

    Minor (compare)

  • Dec 14 06:00
    ejsmith synchronize #452
  • Dec 14 06:00

    ejsmith on elastic7

    Add missing task id (compare)

  • Dec 13 22:43
    ejsmith synchronize #452
  • Dec 13 22:43

    ejsmith on elastic7

    More reindex job changes (compare)

  • Dec 13 15:46
    ejsmith synchronize #452
  • Dec 13 15:46

    ejsmith on elastic7

    Just retry 3 times (compare)

  • Dec 13 15:44
    ejsmith synchronize #452
  • Dec 13 15:44

    ejsmith on elastic7

    More log message changes (compare)

  • Dec 13 14:25
    niemyjski synchronize #452
  • Dec 13 14:25

    niemyjski on elastic7

    Added retry reindex for disconn… (compare)

  • Dec 13 14:09
    niemyjski synchronize #452
  • Dec 13 14:09

    niemyjski on elastic7

    Updated migration to retry sock… (compare)

Eric J. Smith
@ejsmith
not sure
Blake Niemyjski
@niemyjski
looks like it should be
via using block
but I wonder
I triggered this job and hit an exception
and then my work item job isn’t being rehit
it has a default delay
I am seeing an issue where the queue job appears to stop working.
gotta try and figure that out.
Blake Niemyjski
@niemyjski
hmm
whose going to be pull request #200?
@ejsmith exceptionless/Exceptionless#199
this fixes a few issues of which bill ran into and should also help with deduping
well.. noise..
Blake Niemyjski
@niemyjski
if you like it merge it :)
@ejsmith are you going to be on tonight?
We can review my stats stuff
and other pull request stuff :)
Eric J. Smith
@ejsmith
@niemyjski why are you throttling the work item to once every 15 minutes?
Blake Niemyjski
@niemyjski
Missed that
Maybe I should look into finishing the one time job stuff
Get key needs the context
Eric J. Smith
@ejsmith
don’t know what you mean
Blake Niemyjski
@niemyjski
There's a one time job
Eric J. Smith
@ejsmith
yes, that probably needs a good amount of work still.
Blake Niemyjski
@niemyjski
should be super simple
Eric J. Smith
@ejsmith
it needs to persist that info some place.
Blake Niemyjski
@niemyjski
I just looked at the code and converted two of the jobs to the new thing in 30 seconds
cachelockprovider
Eric J. Smith
@ejsmith
can do it in cache, but that doesn’t live forever… but maybe it lives love enough.
Blake Niemyjski
@niemyjski
already have access to work item data
and you pass that to GetKey()
cache lives as long as required?
Eric J. Smith
@ejsmith
probably
it should live for a really long time, but it’s not guaranteed.
Blake Niemyjski
@niemyjski
    public override Task<ILock> GetWorkItemLockAsync(object workItem, CancellationToken cancellationToken = new CancellationToken()) {
        var cacheKey = $"{nameof(RemoveOrganizationWorkItemHandler)}:{((RemoveOrganizationWorkItem)workItem).OrganizationId}";
        return _lockProvider.AcquireAsync(cacheKey, TimeSpan.FromMinutes(15), new CancellationToken(true));
    }
Eric J. Smith
@ejsmith
I’m not sure how the 1 time work item handler would be different than just adding the getworkitemlock
guess it could just be a shortcut that just calls GetKey for you in the GetWorkItemLock
Blake Niemyjski
@niemyjski
yeah
could make these a lot more easier to use if we put a generic on workitemhandler<WorkItemHandlerBase>
but that might make other things harder todo..
Eric J. Smith
@ejsmith
how long would we hold the lock for?
huh?
Blake Niemyjski
@niemyjski
15 minutes or until we release it or renew it
Eric J. Smith
@ejsmith
that isn’t really 1 time.
I think the idea should be that 1 time means 1 time… meaning we set a key that doesn’t expire for like 24 hours or something.
so just do a throttling lock provider for 1 hit every 24 hours
should keep it from happening again.
kind of feel like we should make people implement that themselves.