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

Client API for executing queries and awaiting results #1262

Merged
merged 1 commit into from May 25, 2017

Conversation

3 participants
@jasonmitchell
Contributor

jasonmitchell commented Mar 27, 2017

Reopening #1204 for issue #672

@jasonmitchell jasonmitchell changed the title from Awaiting query result to Client API for executing queries and awaiting results Mar 27, 2017

private async Task<string> GetStatusAsync(string name, UserCredentials userCredentials)
{
var projectionStatus = await _projectionsManager.GetStatusAsync(name, userCredentials);
return projectionStatus.ParseJson<JObject>()["status"].ToString();

This comment has been minimized.

@gregoryyoung

gregoryyoung Apr 3, 2017

Member

We probably want these passed in as timespans

@gregoryyoung

gregoryyoung Apr 3, 2017

Member

We probably want these passed in as timespans

This comment has been minimized.

@jasonmitchell

jasonmitchell Apr 4, 2017

Contributor

Makes sense, I'll make this change.

@jasonmitchell

jasonmitchell Apr 4, 2017

Contributor

Makes sense, I'll make this change.

@pgermishuys

This comment has been minimized.

Show comment
Hide comment
@pgermishuys

pgermishuys Apr 24, 2017

Member

Hi @jasonmitchell, it would seem that a rebase has gone wrong here. We had to repurpose the 4.0.1 release last week and this would likely need to go into 4.0.2. Sorry once again.

Member

pgermishuys commented Apr 24, 2017

Hi @jasonmitchell, it would seem that a rebase has gone wrong here. We had to repurpose the 4.0.1 release last week and this would likely need to go into 4.0.2. Sorry once again.

@jasonmitchell

This comment has been minimized.

Show comment
Hide comment
@jasonmitchell

jasonmitchell Apr 24, 2017

Contributor

No problem, I'll rebase onto 4.0.2 when I'm home from work and update the PR.

Contributor

jasonmitchell commented Apr 24, 2017

No problem, I'll rebase onto 4.0.2 when I'm home from work and update the PR.

@jasonmitchell jasonmitchell changed the base branch from release-v4.0.1 to release-v4.0.2 Apr 24, 2017

@pgermishuys pgermishuys merged commit 4777835 into EventStore:release-v4.0.2 May 25, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
wercker/build-mono4 Wercker pipeline passed
Details

hayley-jean added a commit that referenced this pull request Jul 31, 2017

Merge pull request #1262 from jasonmitchell/awaiting-query-result
Client API for executing queries and awaiting results
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment