Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c1d1e184 authored by Tej Singh's avatar Tej Singh Committed by Automerger Merge Worker
Browse files

Merge "Cleanup appsearch platform logger" into sc-dev am: 5801ebb8 

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

Change-Id: I2270c61f591a7d39fc255f848f875cbf69972d10
parents 5176d2b6 5801ebb8

apex/appsearch/service/java/com/android/server/appsearch/stats/PlatformLogger.java

+4 −4
Original line number Diff line number Diff line
@@ -203,7 +203,7 @@ public final class PlatformLogger implements AppSearchLogger { 
                    stats.getNumOperationsSucceeded(),

                    stats.getNumOperationsFailed());

        } 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,

            //  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.getNativeExceededMaxNumTokens());

        } 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,

            //  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.getResultWithSnippetsCount());

        } 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,

            //  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.

     *

     * <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.

     */

    // TODO(b/173532925) Once we add CTS test for logging atoms and can inspect the result, we can