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

APP-3054: new legacy and BDK2.0 release #211

Merged
merged 4 commits into from
Sep 11, 2020
Merged

APP-3054: new legacy and BDK2.0 release #211

merged 4 commits into from
Sep 11, 2020

Conversation

symphony-elias
Copy link
Contributor

No description provided.

symphony-hong and others added 4 commits September 11, 2020 18:20
* First experiment on Fluent API setup

Create ServiceFactory class for providing services to SymphonyBdk entry point.

Simplify code in SymphonyBdk

Experimental: First implementing OBO call on StreamService.

User can do an OBO authentication by calling SymphonyBdk#obo(Obo.Handle). This method will return an AuthSession which is an obo session.

OboStreamService contains only OBO-enabled stream apis. User will do the OBO authentication out of scope of SymphonyBdk, then will pass the AuthSession which is returned by the previous step to the method to perform the call with oboSession.

* Update @symphony-thibault's comments

* Update ServiceFactory

* Update OBO for user Service

Create OboUserService for calling user service methods using OBO Session

Remove methods wrapping the deprecated user service API

Remove redundant OBO class

* Add javadoc

* Update SymphonyBdk and add unittest

* Update documentation for fluent api

* Update documentation

* Make the datafeed service retry when a SocketTimeoutException was caught
Copy link
Contributor

@symphony-hong symphony-hong left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@symphony-elias symphony-elias merged commit 9b2c6c9 into finos:1.2-rc Sep 11, 2020
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