Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • 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)

  • Dec 13 04:51
  • Dec 13 03:28
    ejsmith synchronize #452
  • Dec 13 03:28

    ejsmith on elastic7

    Oops progress (compare)

  • Dec 13 03:04
    ejsmith synchronize #452
  • Dec 13 03:04

    ejsmith on elastic7

    More log formatting (compare)

  • Dec 13 02:02
    ejsmith synchronize #452
  • Dec 13 02:02

    ejsmith on elastic7

    Fix tests? (compare)

  • Dec 13 01:18
    ejsmith synchronize #452
  • Dec 13 01:18

    ejsmith on elastic7

    Change duration format (compare)

  • Dec 13 00:39
    ejsmith synchronize #452
Sander Rijken
@srijken
we could also do a reader/writer lock
Blake Niemyjski
@niemyjski
what ever you think is best and easiest to maintain :)
Sander Rijken
@srijken
:)
it’s not that much harder
Blake Niemyjski
@niemyjski
@ejsmith this stuff is pretty specific to exceptionless
hitting some roadblocks
Eric J. Smith
@ejsmith
with the aggregations?
stuff like avg:somefield min:somefield, max:somefield
that shouldn’t be specific
Blake Niemyjski
@niemyjski
i know
but we need access to the elastic index for alias name
Eric J. Smith
@ejsmith
so what is specific?
Blake Niemyjski
@niemyjski
can share my screen quick
Eric J. Smith
@ejsmith
what do you mean alias name
can’t do that.
Blake Niemyjski
@niemyjski
No generic way to inject the index on the context unless we add a constraint to the index with a <T> for the hell of it to auto inject it
then the date stuff is not generic but something we’d have to ahve
that’s only for terms tho
Screen Shot 2016-04-11 at 4.12.07 PM.png
            .WithDateRange(utcStart, utcEnd, EventIndex.Fields.PersistentEvent.Date)
            .WithIndices(utcStart, utcEnd, $"'{_eventIndex.VersionedName}-'yyyyMM”); 
is the biggest thing
you want to do smart date ranges
Sander Rijken
@srijken
@niemyjski no ReaderWriterLock in PCL it seems, so just keep the lock the way it is?
Blake Niemyjski
@niemyjski
yeah I guess
Eric J. Smith
@ejsmith
that is up to the user to pass in a filter and system filter just like any other thing in the repo
aggregations are on top of that.
Blake Niemyjski
@niemyjski
wonder if we can get away with non concurrent queues but is it worth it? what might it introduce
Eric J. Smith
@ejsmith
use the exact same query builder stuff
Blake Niemyjski
@niemyjski
yeah
ok
Sander Rijken
@srijken
and, how much do you think we gain?
Blake Niemyjski
@niemyjski
so object query as first param
yeah good idea :)
@srijken I’m not sure if anything but less chances to dead lock???
between there lock implementations and ours but I don’t see how that would happen with our locks being around everything
see first answer
that kinda suggests that it is in PCL?
oh where’s there’s a bunch of platforms
just leave it
could try a normal queue, and see if the tests + your recent problem are still OK?
Eric J. Smith
@ejsmith
really hope the new TFM stuff will make this simpler.
Sander Rijken
@srijken
TFM?
is that missing an R in front? ;)
Eric J. Smith
@ejsmith
lol no
Target Framework Moniker
Sander Rijken
@srijken
right
Eric J. Smith
@ejsmith
the stuff where you target an API surface area and all platforms that support that contract or higher just wokr.
and you don’t have to do stuff for each platform
Sander Rijken
@srijken
@niemyjski merge conflict is fixed