Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • 14:47
    niemyjski synchronize #452
  • 14:47

    niemyjski on elastic7

    add failed task ids to log mess… (compare)

  • 09:40
    zhangyiqiu starred exceptionless/Exceptionless
  • 07:45
  • 05:47
    ejsmith edited #471
  • 05:46
    ejsmith opened #471
  • 05:46

    ejsmith on strip-large-data

    Strip large data values out of … (compare)

  • Dec 11 19:54
    niemyjski synchronize #452
  • Dec 11 19:54

    niemyjski on elastic7

    Decrease batch size (compare)

  • Dec 11 14:51
    niemyjski synchronize #452
  • Dec 11 14:51

    niemyjski on elastic7

    Reset the alias cache before ru… (compare)

  • Dec 11 04:03

    ejsmith on mappings-refactor

    (compare)

  • Dec 11 04:02
    ejsmith synchronize #452
  • Dec 11 04:02

    ejsmith on elastic7

    Change a bunch of index field s… Updated dependencies. Revert to 2201-pre and 6 more (compare)

  • Dec 11 04:02
    ejsmith closed #470
  • Dec 11 04:02
    ejsmith edited #470
  • Dec 11 03:59
    ejsmith opened #470
  • Dec 11 03:14

    niemyjski on mappings-refactor

    Updated to latest foundatio (compare)

  • Dec 11 02:59

    niemyjski on mappings-refactor

    Tweaked the event index mapping… (compare)

  • Dec 11 01:53

    ejsmith on mappings-refactor

    Update cache client list method… (compare)

Sander Rijken
@srijken
for the rest we want ConcurrentDictionary behavior
Blake Niemyjski
@niemyjski
really wish I could step into these extension methods or go to source but all I see is empty definitions
yep
Sander Rijken
@srijken
_mergedEvents.FirstOrDefault() has the same issue
Blake Niemyjski
@niemyjski
yeah
Sander Rijken
@srijken
hmm what to do
Blake Niemyjski
@niemyjski
well I think the issue is that it’s not found in the queue and then other code runs then it enqueues it between checks and the lock prevents that
Sander Rijken
@srijken
the lock makes the whole thing threadsafe yeah
btw, what’s the difference between FormattedInfo and FormattedTrace?
couldn’t get the Trace to show up in the test runner
Blake Niemyjski
@niemyjski
not sure
oh
LogLevel
and there is a min level in the unit tests of Debug
could update that in GetClient
Sander Rijken
@srijken
to get rid of the lock, we would have to keep both a concurrentdictionary (for checking if we have them) and a concurrentqueue (for cleanup of the dictionary)
Blake Niemyjski
@niemyjski
or CreateClient on the plugin tests
well if we are locking do we even need concurrentqueue anymore?
Sander Rijken
@srijken
no
Blake Niemyjski
@niemyjski
and is concurrentdictionary even in the pcl?
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