> On July 30, 2014, 4:25 a.m., David McLaughlin wrote: > > Not sure I understand the motivation here. It's generally best to host your > > own static assets.
Users typically debug their Aurora jobs by looking at individual instances, which usually involves accessing the web interfaces (provided by the Thermos Observer) of several (or numerous) slaves. When users access the web interface of different slaves their browser will download jquery.js from each slave visited, unless it already has it cached (they've previously visited the same slave). The browser's cache hit-rate for jquery.js is practically zero because each slave hosts to its own copy. If we instead point to a version stored on a CDN then it's likely that the browser will already have jquery in its cache, therefore the request can be served immediately, which will make the UI slightly faster and save a small amount of bandwidth. - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23329/#review49084 ----------------------------------------------------------- On July 9, 2014, 9:50 p.m., David Robinson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/23329/ > ----------------------------------------------------------- > > (Updated July 9, 2014, 9:50 p.m.) > > > Review request for Aurora, David McLaughlin and Brian Wickman. > > > Bugs: AURORA-578 > https://issues.apache.org/jira/browse/AURORA-578 > > > Repository: aurora > > > Description > ------- > > remove embedded jquery > > > Diffs > ----- > > src/main/python/apache/thermos/observer/http/templates/filebrowse.tpl > 511d7c06206ae5fd8a4206683f09348e1276b8c4 > src/main/python/apache/thermos/observer/http/templates/index.tpl > 3ccb6e841c932cb8bcb43b765e0b5aa8bc567f88 > src/main/python/apache/thermos/observer/http/templates/logbrowse.tpl > b182a4b331fbe8b9dd437194d195d220184a2f7c > > Diff: https://reviews.apache.org/r/23329/diff/ > > > Testing > ------- > > > Thanks, > > David Robinson > >
