These are chat archives for exceptionless/Discuss

16th
Mar 2016
Eric J. Smith
@ejsmith
Mar 16 2016 04:38
What's up people?
guy from nlog commented in favor of us
they added the ability to do +1 on a post too
kinda cool :D
Bill Henning
@BillHenning
Mar 16 2016 14:23
github rules
Blake Niemyjski
@niemyjski
Mar 16 2016 14:24
now if we can just tell @boydpatterson that :D
Eric J. Smith
@ejsmith
Mar 16 2016 15:03
I can’t believe they didn’t add a ship it reaction.
Blake Niemyjski
@niemyjski
Mar 16 2016 15:27
angular is really pissing me off
so
these input groups put styling and merge button elements that are in the root of the button input group
but with angular you can’t merge components/directives unless it’s under one root element
so I have to wrap it all in a div
to insert it which screws up the styling completely
:(
would be super simple if we didn’t have dropdowns
Eric J. Smith
@ejsmith
Mar 16 2016 15:29
well, we do. :-)
Blake Niemyjski
@niemyjski
Mar 16 2016 15:30
been fighitng it all morning
twbs/bootstrap#14293
wish I could just pay someone on the bootstrap team to fix this in 3 seconds
Eric J. Smith
@ejsmith
Mar 16 2016 15:35
if it makes it easier for you to do a single component that manages all of the filter ui state at the top then do that.
Blake Niemyjski
@niemyjski
Mar 16 2016 15:36
you mean on different sides
?
Eric J. Smith
@ejsmith
Mar 16 2016 15:36
huh?
Blake Niemyjski
@niemyjski
Mar 16 2016 15:36
yeah
Eric J. Smith
@ejsmith
Mar 16 2016 15:36
no I mean make the entire thing one angular component
Blake Niemyjski
@niemyjski
Mar 16 2016 15:36
might have to but that really really sucks
Eric J. Smith
@ejsmith
Mar 16 2016 15:36
its fine… they are all related.
just keep the search service
separate
filter service
Blake Niemyjski
@niemyjski
Mar 16 2016 15:37
yeah it is and has been
just sucks mergeing those
Eric J. Smith
@ejsmith
Mar 16 2016 15:37
this angular component is just for managing the ui
Blake Niemyjski
@niemyjski
Mar 16 2016 15:37
yeah
Eric J. Smith
@ejsmith
Mar 16 2016 15:37
do it
Blake Niemyjski
@niemyjski
Mar 16 2016 15:37
but we might want to reuse these controls some place
Eric J. Smith
@ejsmith
Mar 16 2016 15:37
I want to get that done quickly
then we can reuse the logic
Blake Niemyjski
@niemyjski
Mar 16 2016 15:37
not if they are in a merged control
Eric J. Smith
@ejsmith
Mar 16 2016 15:37
and plus we aren’t using them any other places
Blake Niemyjski
@niemyjski
Mar 16 2016 15:37
ok
Eric J. Smith
@ejsmith
Mar 16 2016 15:37
sure you can
Blake Niemyjski
@niemyjski
Mar 16 2016 15:38
no you can't
not if it’s in one search control
lol
Eric J. Smith
@ejsmith
Mar 16 2016 15:38
make the single angular control a composite
Blake Niemyjski
@niemyjski
Mar 16 2016 15:38
right now I have the date filter seperated
composite?
Eric J. Smith
@ejsmith
Mar 16 2016 15:38
a composite contorl
Blake Niemyjski
@niemyjski
Mar 16 2016 15:38
well I wonder if I can just copy the bootstrap css and modify it to take a div into affect
Eric J. Smith
@ejsmith
Mar 16 2016 15:38
1 control that encapsulates several internal controls
but it doesn’t matter
just get it working
Blake Niemyjski
@niemyjski
Mar 16 2016 15:39
that’s what I’m doing right now nub
lol
Eric J. Smith
@ejsmith
Mar 16 2016 15:39
we aren’t using that
any other places
Blake Niemyjski
@niemyjski
Mar 16 2016 15:45
guess I’ll just merge them all into one control
:(
Blake Niemyjski
@niemyjski
Mar 16 2016 16:03
got it with a css workaround
:D
put the <div class=“inline"
then
div.input-group-btn div.inline:not(:first-child) .btn {
border-radius:0;
margin-left: -3px
}
Screen Shot 2016-03-16 at 11.04.02 AM.png
css selectors FTW
sucks it’s not in the exact expected syntax but angular...
Eric J. Smith
@ejsmith
Mar 16 2016 16:05
something you are doing is causing a gab between the search box and date selection
Blake Niemyjski
@niemyjski
Mar 16 2016 16:05
yeah
the div
Eric J. Smith
@ejsmith
Mar 16 2016 16:05
ok, can probably just change some styles.
leave it
get it functioning and I will try and make it look good. :-)
Blake Niemyjski
@niemyjski
Mar 16 2016 16:05
there is no gab in it
zoomed in 400%
the textbox is a few pixels higher tho
Eric J. Smith
@ejsmith
Mar 16 2016 16:06
I am not sure how we are ging to get the textbox to take the full width.
I think that is going to be tricky
get it working and I will see if some better front end people have ideas on how to do that.
hmm… just noticed that the project dropdown is kind of messed up too… worried you are doing html that will need to change and then your component won’t work.
Blake Niemyjski
@niemyjski
Mar 16 2016 17:24
woot
got it all styled
almost ready to commit
got it full width and looking great even on mobile :)
we need to do something with the user menu on mobile it looks like :poop:
Blake Niemyjski
@niemyjski
Mar 16 2016 18:15
@BillHenning @ejsmith you guys wanted to keep the project filter selection across page load
bit difficult...
almost impossible
because it should redirect to the right view, and right now we pick it up from the url
and search is visible when you are on the my account and org pages
Bill Henning
@BillHenning
Mar 16 2016 18:19
so i guess i could bookmark the page that specifically contains data for the one app
that's probably fine, i hadn't noticed that it changed the URL
i'd rather get stuff like country report anyhow first :)
Blake Niemyjski
@niemyjski
Mar 16 2016 18:20
yeah
in the future that could change… if we just have the 4 simple views and then the filter actually has the project id in the search area..
then it would be a no brainer
Blake Niemyjski
@niemyjski
Mar 16 2016 18:51
/all what do you think of this: https://exceptionless.com/simple-app-deployment-azure-continuous-deployment-github/ anything we should change?
Blake Niemyjski
@niemyjski
Mar 16 2016 18:57
@ejsmith so some feedback on the queue changes
should we be special casing notimplemented exception and not running a retry on it
what do I need to do if I override GetLockAsync.. will that be retried for me or do I need to do something for it?
Eric J. Smith
@ejsmith
Mar 16 2016 18:59
What would be not implemented?
It will be retried
We need to do a blog post dedicated to our redis queue and maybe a video for that.
As far as I'm aware of, we are the only free .net redis queue .
Blake Niemyjski
@niemyjski
Mar 16 2016 19:02
yeah
@joeltjen :)
@jmoeltjen oops :D
Eric J. Smith
@ejsmith
Mar 16 2016 19:04
Might be nice to
Blake Niemyjski
@niemyjski
Mar 16 2016 19:04
I just noticed that our cleanup jobs were getting behind and I wonder if it’s because they are not running that often.. I just bumped them up to every 2 hours
Eric J. Smith
@ejsmith
Mar 16 2016 19:05
I think the cleanup is happening at the same time as the backup.
Blake Niemyjski
@niemyjski
Mar 16 2016 19:05
yeah
@jmoeltjen great job on the blog post :D
so I changed it up a bit
Eric J. Smith
@ejsmith
Mar 16 2016 19:07
I want to do a tour of Foundatio video
Blake Niemyjski
@niemyjski
Mar 16 2016 19:07
yeah that would be cool
sweet
think I found a second issue with our curator script
--timeout 60
helps a lot
might even bump that up to 120
Sander Rijken
@srijken
Mar 16 2016 19:14
hi
Blake Niemyjski
@niemyjski
Mar 16 2016 19:14
hey
how are you doing
http://dev-app.exceptionless.io/ is updating with the new search bar… we still need to do formatting / feedback as well as update the server side to default include fixed and include hidden to false..
Eric J. Smith
@ejsmith
Mar 16 2016 19:26
Yes I did 600
Blake Niemyjski
@niemyjski
Mar 16 2016 19:40
dang
elastic/curator#582
seems like a bigger issue
Sander Rijken
@srijken
Mar 16 2016 20:23
hmm suppose an exception happens that takes down the apppool
it will be stored somewhere right?
I have a catch that rethrows, but calls .ToExceptionless().Submit()
Blake Niemyjski
@niemyjski
Mar 16 2016 20:27
ah
if you aren’t calling useFolderStorage or useIsolatedStorage then no,
it won’t per persisted if the app crashes
we do have code that looks at process close but if it’s a web app I don’t think we can do anything
maybe we could wire up to app domain unload?
unloading
Sander Rijken
@srijken
Mar 16 2016 20:28
so if I configure folder storage in the config, will that fix it?
Blake Niemyjski
@niemyjski
Mar 16 2016 20:29
yeah
and it supports |DataDirectory|
Sander Rijken
@srijken
Mar 16 2016 20:35
is there a delay before it starts processing the queue?
because I see the queue filling, but nothing coming in
Blake Niemyjski
@niemyjski
Mar 16 2016 20:40
@sharathsatish how’s the migration coming along?
10 seconds
unless you call processqueue
then it’s immediate
Sander Rijken
@srijken
Mar 16 2016 20:43
async/await doesn’t really improve readibility of stack huh
Sander Rijken
@srijken
Mar 16 2016 21:00
looks like I fixed the bug, which is a good thing
strange is that after calling ProcessQueue, there’s still 900 items in the queue folder
Eric J. Smith
@ejsmith
Mar 16 2016 21:06
nope, async makes call stacks suck.
I wish they would do something about that.
they can recognize the pattern and transform it.
Sander Rijken
@srijken
Mar 16 2016 21:29
like the async checkbox in chrome dev tools
Eric J. Smith
@ejsmith
Mar 16 2016 21:29
yes, I’ve seen tools that will do it.
Roslyn should be able to do it.
Eric J. Smith
@ejsmith
Mar 16 2016 21:41
ugh, so we need to do some awaits inside of Dispose… I was just using async void knowing that it would be fire and forget and it was mostly working...
but it seems that it doesn’t always get executed.
if I change to use .Wait() it deadlocks.
yay
Sander Rijken
@srijken
Mar 16 2016 21:43
what’s the deal with .json and .json.x in the queue?
x is being processed?
Eric J. Smith
@ejsmith
Mar 16 2016 21:43
yeah, its a lock
Sander Rijken
@srijken
Mar 16 2016 22:08
hmm I still see queue dir filling, and .x things appearing, and nothing new in exceptionless at the site
Eric J. Smith
@ejsmith
Mar 16 2016 22:09
queue is getting stuck?
Sander Rijken
@srijken
Mar 16 2016 22:09
dunno yet
Sander Rijken
@srijken
Mar 16 2016 22:20
enabled logging.. but no logfile
Eric J. Smith
@ejsmith
Mar 16 2016 22:23
This on the .net client?
Sander Rijken
@srijken
Mar 16 2016 22:23
yes
Eric J. Smith
@ejsmith
Mar 16 2016 22:25
Did you enable logging and set it to use the file logger?
Sander Rijken
@srijken
Mar 16 2016 22:25
nevermind
Eric J. Smith
@ejsmith
Mar 16 2016 22:25
?
Sander Rijken
@srijken
Mar 16 2016 22:25
I just set logPath
Eric J. Smith
@ejsmith
Mar 16 2016 22:26
So you got it to work?
Sander Rijken
@srijken
Mar 16 2016 22:26
I’m sure if I add enableLogging=“true” it’ll work
sometimes I feel so stupid :)
Eric J. Smith
@ejsmith
Mar 16 2016 22:27
Setting the log path and not having enableLogging false should turn it on automatically. But it might not be doing that.
Haha i know that feeling.
Sander Rijken
@srijken
Mar 16 2016 22:28
well the wiki tells me to add enableLogging=“true” as well
Eric J. Smith
@ejsmith
Mar 16 2016 22:28
Async deadlocks make me feel really dumb
Sander Rijken
@srijken
Mar 16 2016 22:28
yeah
well.. async makes me feel dumb
Eric J. Smith
@ejsmith
Mar 16 2016 22:30
Me too. LOL
apparently just having “enableLog=true” will work
and leaving out the logpath
Eric J. Smith
@ejsmith
Mar 16 2016 22:31
Yeah that will log in memory.
Sander Rijken
@srijken
Mar 16 2016 22:31
well no
it depends
I have storagePath set, so it would log in the same folder there
Eric J. Smith
@ejsmith
Mar 16 2016 22:32
Yeah, looking.