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.

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

fix: pgsql driver check local type error #2192

Merged
merged 1 commit into from
Oct 11, 2022
Merged

Conversation

yss930819
Copy link
Contributor

由于驱动内类型检查函数名错误,导致未能使用正确类型生成 entry 模型

由于驱动内类型检查函数名错误,导致未能使用正确类型生成 entry 模型
@codecov-commenter
Copy link

Codecov Report

Base: 76.49% // Head: 76.50% // Increases project coverage by +0.01% 🎉

Coverage data is based on head (e0b03c4) compared to base (847f016).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2192      +/-   ##
==========================================
+ Coverage   76.49%   76.50%   +0.01%     
==========================================
  Files         580      577       -3     
  Lines       48176    47838     -338     
==========================================
- Hits        36853    36600     -253     
+ Misses       9304     9231      -73     
+ Partials     2019     2007      -12     
Flag Coverage Δ
go-1.15-386 ?
go-1.15-amd64 ?
go-1.16-386 ?
go-1.16-amd64 76.50% <100.00%> (+0.01%) ⬆️
go-1.17-386 ?
go-1.17-amd64 ?
go-1.18-386 ?
go-1.18-amd64 ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
contrib/drivers/pgsql/pgsql.go 64.28% <100.00%> (+3.06%) ⬆️
os/gcron/gcron_schedule_fix.go 87.50% <0.00%> (-12.50%) ⬇️
os/gspath/gspath_cache.go 84.21% <0.00%> (-10.53%) ⬇️
os/gfsnotify/gfsnotify_watcher_loop.go 75.63% <0.00%> (-7.57%) ⬇️
net/gtcp/gtcp_pool.go 68.18% <0.00%> (-6.82%) ⬇️
os/gfsnotify/gfsnotify_filefunc.go 70.12% <0.00%> (-3.90%) ⬇️
container/gmap/gmap_hash_str_any_map.go 97.00% <0.00%> (-1.00%) ⬇️
contrib/config/kubecm/kubecm.go
contrib/config/kubecm/kubecm_kube.go
contrib/drivers/oracle/oracle.go
... and 1 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@gqcn gqcn merged commit b96b5c3 into gogf:master Oct 11, 2022
houseme added a commit to houseme/gf that referenced this pull request Oct 13, 2022
* master: (29 commits)
  fix issue gogf#1946 (gogf#2194)
  fix issue of OmitEmptyWhere in Builder for package gdb (gogf#2195)
  fix: modify `Polaris` config readme.md (gogf#2186)
  fix info content when listens on port :0 for ghttp.Server (gogf#2191)
  fix: pgsql driver check local type error (gogf#2192)
  new version v2.2.0 (gogf#2185)
  feat: temporarily disable the unit testing of the Polaris configuration center (gogf#2183)
  package comments and readme update (gogf#2182)
  feat: create polaris config (gogf#2170)
  add function `ZipPathContent` for package `gcompress` (gogf#2179)
  feat: improve glog for polaris  register (gogf#2178)
  improve port listening for ghttp.Server (gogf#2175)
  add WithUUID for package gtrace (gogf#2176)
  fix issue gogf#1965 (gogf#2177)
  fix issue gogf#1965 (gogf#2174)
  fix issue gogf#2172 (gogf#2173)
  add `gcfg.Adapter` implements using apollo service (gogf#2165)
  add watch feature for package kubecm (gogf#2164)
  fix configuration management for package gdb (gogf#2163)
  add local db configuration support for package gdb (gogf#2161)
  ...

# Conflicts:
#	contrib/config/apollo/README.MD
#	contrib/config/apollo/apollo.go
#	contrib/config/kubecm/README.MD
#	contrib/config/kubecm/kubecm.go
#	contrib/drivers/README.MD
#	contrib/registry/polaris/go.mod
#	contrib/registry/polaris/go.sum
#	database/gdb/gdb_driver_wrapper_db.go
#	example/go.mod
#	example/go.sum
#	version.go
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.

None yet

3 participants