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

Add account level metrics control #1462

Merged
merged 5 commits into from
Feb 9, 2022

Conversation

And1sS
Copy link
Member

@And1sS And1sS commented Sep 8, 2021

No description provided.

@And1sS And1sS added the work in progress Signals not finished work label Sep 8, 2021
@muuki88
Copy link
Contributor

muuki88 commented Sep 13, 2021

If I understand this correctly there are now two ways to control the verbosity, right? Will the old be deprecated?

Also an example on how this looks and a change to the docs would be super helpful ❤️

…etrics-control

# Conflicts:
#	src/main/java/org/prebid/server/auction/ExchangeService.java
@And1sS And1sS removed the request for review from rpanchyk February 4, 2022 12:43
@And1sS
Copy link
Member Author

And1sS commented Feb 4, 2022

If I understand this correctly there are now two ways to control the verbosity, right? Will the old be deprecated?

Also an example on how this looks and a change to the docs would be super helpful heart

Hi, @muuki88, I finally managed to return back to this pr, and updated docs. Is everything clear, any suggestions?
This will just add a new way to controll verbosity for concrete account, wich will override the old one.

@And1sS And1sS removed the work in progress Signals not finished work label Feb 4, 2022
@And1sS And1sS requested a review from Net-burst February 4, 2022 12:46
@SerhiiNahornyi SerhiiNahornyi merged commit 6c62fb8 into master Feb 9, 2022
@SerhiiNahornyi SerhiiNahornyi deleted the add-account-level-metrics-control branch February 9, 2022 18:51
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.

3 participants