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

RTD framework #2104

Closed
wants to merge 1 commit into from
Closed

Conversation

TechDevCrew
Copy link

Dear community!

Could you please give your attention at my draft pull request:

I'm trying to implement RTD framework according this issue.
Does it correct way of implementation?
Please provide your suggestions, considerations or any help with developing this feature.

@TechDevCrew TechDevCrew changed the title feature rtd framework RTD framework Dec 8, 2021
@SyntaxNode
Copy link
Contributor

Thank you for opening this PR. I appreciate your interest to build this feature.

Building a new extension to Prebid Server is typically a large undertaking. The configuration, discovery of modules, runtime, handling of panics and errors, protection of shared memory, metrics, and logs are all factors which need to be considered. This also might make more sense to wait for #1734 to be implemented first.

The Prebid Server committee has not yet reviewed your request to add RTD features on the server side. The responses you've received in the issue are the preliminary personal opinions of Bret. The committee is next meeting on December 10, 2021. We'll review your request then and comment on the issue.

@SyntaxNode
Copy link
Contributor

SyntaxNode commented Dec 17, 2021

The Prebid Server Committee discussed RTD functionality and we agree with @bretg that this would be best suited for after the Modularity Framework. The hooks described in that specification are very likely sufficient for RTD if we add helper methods or framework for buffering and sending data to RTD servers.

The specification for Modularity has already been through review and is implemented by PBS-Java. We will begin to implement it in PBS-Go early next year in 2022. I'm closing this PR in favor of Modularity and you are absolutely welcomed to help us code it. If you would like, please coordinate with us on the Modularity Framework issue.

Thank you in advance for your understanding.

@SyntaxNode SyntaxNode closed this Dec 17, 2021
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.

2 participants