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

qstream api isn't affected by &length parameter #266

Closed
Ivan-Shestakov opened this issue Oct 31, 2017 · 0 comments
Closed

qstream api isn't affected by &length parameter #266

Ivan-Shestakov opened this issue Oct 31, 2017 · 0 comments
Labels
bug

Comments

@Ivan-Shestakov
Copy link

When running this query
curl -v 'http://cm-well-host.com/?op=qstream&length=1000&format=ntriples&with-data&with-system'
the stream runs indefinitely.

@hochgi found there's unimplemented TODO for this:
https://github.com/thomsonreuters/CM-Well/blob/master/server/cmwell-ws/app/controllers/Application.scala#L915-L916
hochgi pushed a commit to hochgi/CM-Well that referenced this issue Nov 12, 2017
CM-Well#266 - length in qstream + boyscouting
0986539
hochgi pushed a commit that referenced this issue Nov 12, 2017
Merge pull request #285 from hochgi/cmwell-266-qstream-length
11eea67 
#266 - length in qstream + boyscouting
@hochgi hochgi closed this as completed Nov 29, 2017
e-orz pushed a commit to e-orz/CM-Well that referenced this issue Feb 6, 2021
CM-Well#266 - length in qstream + boyscouting
fc3cc4b
e-orz pushed a commit to e-orz/CM-Well that referenced this issue Feb 6, 2021
Merge pull request CM-Well#285 from hochgi/cmwell-266-qstream-length
10fefe7 
CM-Well#266 - length in qstream + boyscouting
e-orz pushed a commit to e-orz/CM-Well that referenced this issue Feb 6, 2021
CM-Well#266 - length in qstream + boyscouting
70eecbb
e-orz pushed a commit to e-orz/CM-Well that referenced this issue Feb 6, 2021
Merge pull request CM-Well#285 from hochgi/cmwell-266-qstream-length
1cf1717 
CM-Well#266 - length in qstream + boyscouting
e-orz pushed a commit to e-orz/CM-Well that referenced this issue Feb 6, 2021
CM-Well#266 - length in qstream + boyscouting
ef2beb8
e-orz pushed a commit to e-orz/CM-Well that referenced this issue Feb 6, 2021
Merge pull request CM-Well#285 from hochgi/cmwell-266-qstream-length
6d08a94 
CM-Well#266 - length in qstream + boyscouting
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@hochgi @Ivan-Shestakov