Skip to content

Commit 2d2b6a2

Browse files
authored
add directory listing statistics (#4315)
1 parent 4468758 commit 2d2b6a2

File tree

2 files changed

+9
-9
lines changed

2 files changed

+9
-9
lines changed

opengrok-web/src/main/webapp/list.jsp

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,6 @@ java.net.URLEncoder,
3131
java.nio.charset.StandardCharsets,
3232
java.util.List,
3333
java.util.Locale,
34-
java.util.logging.Logger,
3534
java.util.Set,
3635
org.opengrok.indexer.analysis.AnalyzerGuru,
3736
org.opengrok.indexer.analysis.Definitions,
@@ -40,21 +39,18 @@ org.opengrok.indexer.analysis.AnalyzerFactory,
4039
org.opengrok.indexer.analysis.NullableNumLinesLOC,
4140
org.opengrok.indexer.history.Annotation,
4241
org.opengrok.indexer.index.IndexDatabase,
43-
org.opengrok.indexer.logger.LoggerFactory,
4442
org.opengrok.indexer.search.DirectoryEntry,
4543
org.opengrok.indexer.util.FileExtraZipper,
4644
org.opengrok.indexer.util.IOUtils,
4745
org.opengrok.web.DirectoryListing"
4846
%>
4947
<%@ page import="static org.opengrok.web.PageConfig.DUMMY_REVISION" %>
5048
<%@ page import="static org.opengrok.indexer.history.LatestRevisionUtil.getLatestRevision" %>
51-
<%@ page import="org.opengrok.indexer.web.SortOrder" %>
5249
<%@ page import="jakarta.servlet.http.Cookie" %>
5350
<%@ page import="java.util.stream.Collectors" %>
54-
<%@ page import="org.opengrok.indexer.configuration.PathAccepter" %>
55-
<%@ page import="org.opengrok.indexer.configuration.RuntimeEnvironment" %>
56-
<%@ page import="java.text.Format" %>
57-
<%@ page import="java.text.SimpleDateFormat" %>
51+
<%@ page import="org.opengrok.indexer.util.Statistics" %>
52+
<%@ page import="org.opengrok.indexer.logger.LoggerFactory" %>
53+
<%@ page import="java.util.logging.Logger" %>
5854
<%
5955
{
6056
// need to set it here since requesting parameters
@@ -123,6 +119,8 @@ document.pageReady.push(function() { pageReadyList();});
123119
<%
124120
/* ---------------------- list.jsp start --------------------- */
125121
{
122+
final Logger LOGGER = LoggerFactory.getLogger(getClass());
123+
126124
PageConfig cfg = PageConfig.get(request);
127125
String rev = cfg.getRequestedRevision();
128126
Project project = cfg.getProject();
@@ -135,6 +133,8 @@ document.pageReady.push(function() { pageReadyList();});
135133
String rawPath = request.getContextPath() + Prefix.DOWNLOAD_P + path;
136134
Reader r = null;
137135
if (cfg.isDir()) {
136+
Statistics statistics = new Statistics();
137+
138138
// valid resource is requested
139139
// mast.jsp assures, that resourceFile is valid and not /
140140
// see cfg.resourceNotAvailable()
@@ -208,6 +208,8 @@ document.pageReady.push(function() { pageReadyList();});
208208
209209
}
210210
}
211+
212+
statistics.report(LOGGER, Level.FINE, "directory listing done", "dir.list.latency");
211213
} else if (rev.length() != 0) {
212214
// requesting a revision
213215
File xrefFile;

opengrok-web/src/main/webapp/xref.jspf

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,8 +52,6 @@ org.opengrok.indexer.web.QueryParameters"
5252
Genre g = AnalyzerGuru.getGenre(a);
5353
String error = null;
5454

55-
final Logger LOGGER = LoggerFactory.getLogger(getClass());
56-
5755
if (g == Genre.PLAIN || g == Genre.HTML || g == null) {
5856
InputStream in = null;
5957
File tempf = null;

0 commit comments

Comments
 (0)