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 am: 5801ebb8e6

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15238311

Change-Id: I973aff3d09da76778d1ab2d0774d13f3387d88cd
This commit is contained in:
Tej Singh
2021-07-09 08:41:13 +00:00
committed by Automerger Merge Worker
parent b77dbe1e21 5801ebb8e6
commit 82015d860c
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