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

[MDAPI-79] [C++] Retrieve promise-based events from feed #59

Merged
merged 7 commits into from
Oct 28, 2024

Conversation

ttldtor
Copy link
Collaborator

@ttldtor ttldtor commented Oct 17, 2024

  • Added PromiseList<E> class where E - event type: a list of event receiving results that will be completed normally or exceptionally in the future. It is a std::vector<Promise<std::shared_ptr<E>>> wrapper with Graal semantics.
  • Added Promise<void> class.
  • Added Promises class: utility methods to manipulate Promise<> promises.
    • Added Promises::allOf(Collection &&collection) method that returns a new promise that completes when all promises from the given collection complete normally or exceptionally.
  • Fixed the Promise<std::shared_ptr<E>> semantics.
  • Added DXFeed::getLastEventPromise method.
  • Added DXFeed::getLastEventsPromises method.
  • Added DXFeed::getIndexedEventsPromise method.
  • [BREAKING] The DXFeed::getTimeSeriesPromise now returns std::shared_ptr<Promise<std::vector<std::shared_ptr<E>>>>
  • [BREAKING] The IndexedEventSubscriptionSymbol::getSource method now returns std::unique_ptr<IndexedEventSource>

AnatolyKalin and others added 7 commits October 14, 2024 17:30
Promises::allOf
Promise<void>
PromiseList
GraalListWrapper
isolated::promise::IsolatedPromise::allOf
DXFeed::getLastEventPromise -> std::shared_ptr<Promise<std::shared_ptr<E>>>
DXFeed::getIndexedEventsPromise -> std::shared_ptr<Promise<std::vector<std::shared_ptr<E>>>>
DXFeed::getTimeSeriesPromise -> std::shared_ptr<Promise<std::vector<std::shared_ptr<E>>>>
@ttldtor ttldtor marked this pull request as ready for review October 18, 2024 15:04
@AnatolyKalin AnatolyKalin merged commit a733cbc into main Oct 28, 2024
21 checks passed
@AnatolyKalin AnatolyKalin deleted the feature/MDAPI-79 branch October 28, 2024 08:38
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