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.

Sign up for GitHub

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

Jump to bottom

Data explorer - close cursor #1569

Closed
neumino opened this issue Oct 24, 2013 · 3 comments
Closed

Data explorer - close cursor #1569

neumino opened this issue Oct 24, 2013 · 3 comments
Assignees
@neumino
Labels
cp:web-ui
Milestone
1.12

Comments

@neumino
Copy link
Member

neumino commented Oct 24, 2013

We should close the cursor every time we execute a new query.
@ghost ghost assigned neumino Oct 24, 2013
@coffeemug
Copy link
Contributor

Moving to subsequent, since I wouldn't hold the release for this. (I do think we should get it in for 1.11 though)

neumino added a commit that referenced this issue Dec 3, 2013
@neumino
Close cursor + Remove old references to @cursor (it's @state.cursor n…
4b7bd19 
…ow) -- Related to #1569
@neumino
Copy link
Member Author

neumino commented Dec 3, 2013

In review 1067, assigned to @mglukhovsky

@neumino
Copy link
Member Author

neumino commented Dec 7, 2013

Review done.
Merged in next as 9012687

@neumino neumino closed this as completed Dec 7, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@neumino neumino

Labels
cp:web-ui
Projects
None yet
Milestone
1.12
Development

No branches or pull requests
2 participants
@coffeemug @neumino