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

Commit 82015d86 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: I973aff3d09da76778d1ab2d0774d13f3387d88cd
parents b77dbe1e 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