Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Comments

proxy: add hint for streaming fetch #619#621

Merged
bohutang merged 1 commit intoradondb:masterfrom
zhyass:feature_show
Apr 22, 2020
Merged

proxy: add hint for streaming fetch #619#621
bohutang merged 1 commit intoradondb:masterfrom
zhyass:feature_show

Conversation

@zhyass
Copy link
Contributor

@zhyass zhyass commented Apr 22, 2020

[summary]
Add hint for streaming fetch.
select /*+ streaming */
[test case]
src/proxy/query_test.go
[patch codecov]
src/proxy/query.go 88.6%

[summary]
Add hint for streaming fetch.
select /*+ streaming */
[test case]
src/proxy/query_test.go
[patch codecov]
src/proxy/query.go 88.6%
@codecov
Copy link

codecov bot commented Apr 22, 2020

Codecov Report

Merging #621 into master will increase coverage by 0%.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #621   +/-   ##
======================================
  Coverage      89%     89%           
======================================
  Files         128     128           
  Lines       10206   10214    +8     
======================================
+ Hits         9151    9161   +10     
+ Misses        712     711    -1     
+ Partials      343     342    -1     
Impacted Files Coverage Δ
src/proxy/query.go 86% <100%> (+<1%) ⬆️
src/proxy/sessions.go 88% <0%> (+1%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6c1e734...eb7419d. Read the comment docs.

@bohutang
Copy link
Contributor

LGTM

@bohutang bohutang merged commit 15b0251 into radondb:master Apr 22, 2020
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