Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Cleanup appsearch platform logger" into sc-dev

Browse Source
This commit is contained in:
Tej Singh
2021-07-09 08:26:21 +00:00
committed by Android (Google) Code Review
parent 929e584dce 7961d27429
commit 5801ebb8e6
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
apex/appsearch/service/java/com/android/server/appsearch/stats/PlatformLogger.java
View File

@@ -203,7 +203,7 @@ public final class PlatformLogger implements AppSearchLogger {
stats.getNumOperationsSucceeded(), stats.getNumOperationsSucceeded(),
stats.getNumOperationsFailed()); stats.getNumOperationsFailed());
} catch (NoSuchAlgorithmException | UnsupportedEncodingException e) { } catch (NoSuchAlgorithmException | UnsupportedEncodingException e) {
// TODO(b/184204720) report hashing error to Westworld // TODO(b/184204720) report hashing error to statsd
// We need to set a special value(e.g. 0xFFFFFFFF) for the hashing of the database, // We need to set a special value(e.g. 0xFFFFFFFF) for the hashing of the database,
// so in the dashboard we know there is some error for hashing. // so in the dashboard we know there is some error for hashing.
// //
@@ -240,7 +240,7 @@ public final class PlatformLogger implements AppSearchLogger {
stats.getNativeNumTokensIndexed(), stats.getNativeNumTokensIndexed(),
stats.getNativeExceededMaxNumTokens()); stats.getNativeExceededMaxNumTokens());
} catch (NoSuchAlgorithmException | UnsupportedEncodingException e) { } catch (NoSuchAlgorithmException | UnsupportedEncodingException e) {
// TODO(b/184204720) report hashing error to Westworld // TODO(b/184204720) report hashing error to statsd
// We need to set a special value(e.g. 0xFFFFFFFF) for the hashing of the database, // We need to set a special value(e.g. 0xFFFFFFFF) for the hashing of the database,
// so in the dashboard we know there is some error for hashing. // so in the dashboard we know there is some error for hashing.
// //
@@ -286,7 +286,7 @@ public final class PlatformLogger implements AppSearchLogger {
stats.getDocumentRetrievingLatencyMillis(), stats.getDocumentRetrievingLatencyMillis(),
stats.getResultWithSnippetsCount()); stats.getResultWithSnippetsCount());
} catch (NoSuchAlgorithmException | UnsupportedEncodingException e) { } catch (NoSuchAlgorithmException | UnsupportedEncodingException e) {
// TODO(b/184204720) report hashing error to Westworld // TODO(b/184204720) report hashing error to statsd
// We need to set a special value(e.g. 0xFFFFFFFF) for the hashing of the database, // We need to set a special value(e.g. 0xFFFFFFFF) for the hashing of the database,
// so in the dashboard we know there is some error for hashing. // so in the dashboard we know there is some error for hashing.
// //
@@ -363,7 +363,7 @@ public final class PlatformLogger implements AppSearchLogger {
/** /**
* Creates {@link ExtraStats} to hold additional information generated for logging. * Creates {@link ExtraStats} to hold additional information generated for logging.
* *
* <p>This method is called by most of logToWestworldLocked functions to reduce code * <p>This method is called by most of logStatsImplLocked functions to reduce code
* duplication. * duplication.
*/ */
// TODO(b/173532925) Once we add CTS test for logging atoms and can inspect the result, we can // TODO(b/173532925) Once we add CTS test for logging atoms and can inspect the result, we can