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

[feature] add hint for streaming fetch #619

Closed
BohuTANG opened this issue Apr 21, 2020 · 2 comments
Closed

[feature] add hint for streaming fetch #619

BohuTANG opened this issue Apr 21, 2020 · 2 comments
Assignees
@zhyass
Labels
P0 High priority type: feature
Milestone
v1.1.0

Comments

@BohuTANG
Copy link
Contributor

BohuTANG commented Apr 21, 2020 •
edited by zhyass
Loading

Summary

For many drivers with connection pools, the set variable may not work, so we should back to the hints mode:

select /*+ streaming */
@BohuTANG BohuTANG added type: feature P0 High priority labels Apr 21, 2020
@BohuTANG BohuTANG added this to the v1.1.0 milestone Apr 21, 2020
@zhyass
Copy link
Contributor

zhyass commented Apr 21, 2020 •
edited
Loading

ACK

select /*+ streaming */

@BohuTANG BohuTANG changed the title [feature] add hint for streaing fetch [feature] add hint for streaming fetch Apr 21, 2020
@BohuTANG
Copy link
Contributor Author

ACK

select /*+ streaming */

Updated

zhyass added a commit to zhyass/radon that referenced this issue Apr 22, 2020
@zhyass
proxy: add hint for streaming fetch radondb#619
eb7419d 
[summary]
Add hint for streaming fetch.
select /*+ streaming */
[test case]
src/proxy/query_test.go
[patch codecov]
src/proxy/query.go 88.6%
BohuTANG added a commit that referenced this issue Apr 22, 2020
@BohuTANG
Merge pull request #621 from zhyass/feature_show
15b0251 
proxy: add hint for streaming fetch #619
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zhyass zhyass

Labels
P0 High priority type: feature
Projects
None yet
Milestone
v1.1.0
Development

No branches or pull requests
2 participants
@BohuTANG @zhyass