I annotated the bug with the original email description of the fix and
some notes on why I think it's a good idea.
-Max
Benjamin Shine wrote:
Wow, you seem to have really figured out what's going on here, with the
tools and everything.
A few comments:
It sounds like you have identified a performance problem, which this
changeset fixes. Can you please edit
http://jira.openlaszlo.org/jira/browse/LPP-3027 with instructions on how
to reproduce the slow case?
Please put your explanation of what the code does and why into the
change description itself. That way it is stored permanently and easily
connected to those code changes.
How would I use this new dependency checking option? In the "Tests:"
section of the change description, please list either URLs on localhost
we can visit to test the behavior, or a command to execute at the
prompt. The reviewer should be able to run those tests to verify that
the change works as advertised.
For the code itself -- what is the difference between rootDir and
publicRootDir?
+ private transient String publicRootDir;
+ private transient String rootDir;
Thanks for the change, and for using the tools.
-ben
On Nov 7, 2006, at 2:14 PM, Max Carlson wrote:
Sorry, I need to have someone else review the bug before approving.
Henry, can you do a review? Thanks!
I've filed a bug here, to track it
http://jira.openlaszlo.org/jira/browse/LPP-3027
-Max
Naoki Kurosawa wrote:
Hi Max,
I made a patch to add new dependency checking option. New
'checkExceptLPSDir' option checks all user's source file except lps
directory and children.
I think this option is useful for unarchived deploy. (Web application
is directory, not archived WAR) If we use unarchived deploy, cache is
recycled. So we have to use 'check' option. But 'check' option is a
little slow.
Actually, we should use 'never' option with clearing cache per deploy.
But average user don't care about it, and they fall a prey to pit.
In this context, this option may be a pit of the same kind, likes "Why
isn't this updated?" So I don't think we should make this default.
Here is a result of performance test.
(Pentium M 2GHz, MEM:1GB,
Server and client are on same machine. Therefore, I'd like you to
understand this is a rough test.)
/my-apps/copy-of-hello.lzx?lzt=swf
request/sec
check: 13.7
checkExceptLPSDir: 182.6
never: 207.0
with If-Modified-Since header (network traffic is very low)
check: 23.3
checkExceptLPSDir: 341.8
never: 484.1
/demos/weather/weather.lzx?lzt=swf
request/sec
check: 8.6
checkExceptLPSDir: 107.8
never: 139.2
with If-Modified-Since header
check: 14.5
checkExceptLPSDir: 238.7
never: 478.3
/demos/dashboard/dashboard.lzx?lzt=swf
request/sec
check: 3.4
checkExceptLPSDir: 13.1
never: 57.3
with If-Modified-Since header
check: 5.6
checkExceptLPSDir: 25.1
never: 485.2
It is slow when files are many :)
Regards,
Naoki Kurosawa
[EMAIL PROTECTED]
<20061103-naoki-0.tar.gz>
Benjamin Shine
Software Engineer, Open Laszlo / Laszlo Systems
[EMAIL PROTECTED]
