Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RUMM-2170 Create Feature configuration interfaces #933

Merged

Conversation

xgouchet
Copy link
Collaborator

What does this PR do?

Create the interfaces for the SDKv2 features configuration

  • renaming to match the RFC

@xgouchet xgouchet requested a review from a team as a code owner May 19, 2022 05:44
Copy link
Collaborator

@mariusc83 mariusc83 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@0xnm 0xnm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! I left few suggestions for the javadocs.

* Contains the storage configuration for an [FeatureScope] instance.
* @property maxItemSize the maximum size (in bytes) for a single item in a batch
* @property maxItemPerBatch the maximum number of individual items in a batch
* @property maxBatchSize the maximum size of a complete batch
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* @property maxBatchSize the maximum size of a complete batch
* @property maxBatchSize the maximum size (in bytes) of a complete batch

* @property maxItemSize the maximum size (in bytes) for a single item in a batch
* @property maxItemPerBatch the maximum number of individual items in a batch
* @property maxBatchSize the maximum size of a complete batch
* @property oldBatchThreshold the duration after which a batch is considered too old to be
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

better to specify the unit

package com.datadog.android.v2.api

/**
* Describes the format of a paylod for an instance of [SDKFeature].
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* Describes the format of a paylod for an instance of [SDKFeature].
* Describes the format of a payload for an instance of [SDKFeature].

@xgouchet xgouchet force-pushed the xgouchet/RUMM-2170/SDK_Feature_Configuration branch from 1e4e34c to 3349370 Compare May 20, 2022 06:52
@xgouchet xgouchet merged commit e627335 into feature/sdkv2 May 20, 2022
@xgouchet xgouchet deleted the xgouchet/RUMM-2170/SDK_Feature_Configuration branch May 20, 2022 08:54
@xgouchet xgouchet added this to the 1.16.0 milestone Dec 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants