http://coldfusion.com logo
Join Slack
Powered by
Hi, just a note about Coldbox 7: We have updated p...
# box-products
d
Hi, just a note about Coldbox 7: We have updated prod to Coldbox 7 today and we have noticed that the site was slower. After digging into it, the log about the deprecation of renderView hit us in the performance of the site (we have not changed 
renderview
to 
view
yet). After commented out the lines in 
coldbox/system/web/Renderer.cfc
the sites becomes a lot faster.
l
Yes, because, the 
renderView()
is deprecated
and it logs a warnng
every time you use it 🙂
it was meant to slow you down
so you update your code to 
view()
and 
layout()
🙂
a
Yup - I did this. BTW 
view
works in ColdBox 6 so you can do the change before you do the ColdBox 7 update.
d
yep, I kwown 🙂 but I didn't think that that a deprecation would hits the performance so badly (we use renderview a lot)
well maybe a warning about that in the release note could not be a bad idea. Just: "be a good boy and replace to 
view
or your performance will be affected!" 🙂
l
jajaja
good point
same goes for 
announceInterception
image.png
There you go @davidsf
👍 1
r
nice 👌
b
@lmajano When I built warning logs into CommandBox for ModCFML stuff which could happen every single request, I put in logic that would only write each unique log message once to prevent log files from filling up over warnings. Honestly, this is a potential DOS issue.
If you just REALLY want the users to see the message, then throw an error until they activate a specific 
unlockSettingThatWillGoAwaySoon
flag. Otherwise, there's no point in deprecating it if it's basically unusable in production.
l
I’ts usable
it’s just slower
plus, it depends on the log appenders you are using
Plus, David’s app has tons of renderings and traffic
d
just for illustrate the impact in the speed of our home page (more than double):
After commenting out the log lines:
Open in Slack
PreviousNext
Powered by