Gleipnir/nidus-sync
Gleipnir/nidus-sync
7
0
Fork 0
Code Issues 54 Pull requests 4 Projects Releases Packages Wiki Activity Actions

Migrate away from bob as a SQL generation layer #32

New issue
Closed
opened 2026-05-26 18:50:53 +00:00 by eliribble · 3 comments
eliribble commented 2026-05-26 18:50:53 +00:00
Owner
Copy link

Previously I investigated using Bob. I ran into major limitations related to working with PostGIS data. We've been moving toward go-jet, but right now we are using both. I'm suspicious that #3 is at least partially caused or obscured by the double database pooling system that comes with using bob.

Let's finish getting off of it.

We currently have bob-related imports in:

  • api/configuration.go
  • db/connection.go
  • db/fieldseeker.go
  • db/models/*
  • db/pgdata.go
  • db/prepared.go
  • db/sql/*
  • platform/arcgis.go
  • platform/compliance.go
  • platform/csv/csv.go
  • platform/csv/flyover.go
  • platform/csv/pool.go
  • platform/district.go
  • platform/email/template.go
  • platform/fieldseeker.go
  • platform/fieldseeker/point_location.go
  • platform/geocode/geocode.go
  • platform/lead.go
  • platform/notification.go
  • platform/organization.go
  • platform/parcel.go
  • platform/pool.go
  • platform/publicreport/compliance.go
  • platform/publicreport/image.go
  • platform/publicreport/log.go
  • platform/publicreport/nuisance.go
  • platform/publicreport/report.go
  • platform/review.go
  • platform/service_request.go
  • platform/signal.go
  • platform/site.go
  • platform/sync.go
  • platform/text/llm.go
  • platform/tile.go
  • platform/types/address.go
  • platform/upload.go
  • platform/user.go
  • resource/lob_hook.go
  • resource/review_task.go
Previously I investigated using Bob. I ran into major limitations related to working with PostGIS data. We've been moving toward go-jet, but right now we are using both. I'm suspicious that #3 is at least partially caused or obscured by the double database pooling system that comes with using bob. Let's finish getting off of it. We currently have bob-related imports in: - [ ] api/configuration.go - [ ] db/connection.go - [ ] db/fieldseeker.go - [ ] db/models/* - [ ] db/pgdata.go - [ ] db/prepared.go - [ ] db/sql/* - [x] platform/arcgis.go - [x] platform/compliance.go - [ ] platform/csv/csv.go - [ ] platform/csv/flyover.go - [ ] platform/csv/pool.go - [ ] platform/district.go - [x] platform/email/template.go - [ ] platform/fieldseeker.go - [ ] platform/fieldseeker/point_location.go - [ ] platform/geocode/geocode.go - [ ] platform/lead.go - [ ] platform/notification.go - [ ] platform/organization.go - [ ] platform/parcel.go - [ ] platform/pool.go - [ ] platform/publicreport/compliance.go - [ ] platform/publicreport/image.go - [ ] platform/publicreport/log.go - [ ] platform/publicreport/nuisance.go - [ ] platform/publicreport/report.go - [ ] platform/review.go - [ ] platform/service_request.go - [ ] platform/signal.go - [ ] platform/site.go - [ ] platform/sync.go - [ ] platform/text/llm.go - [ ] platform/tile.go - [ ] platform/types/address.go - [ ] platform/upload.go - [ ] platform/user.go - [ ] resource/lob_hook.go - [ ] resource/review_task.go
eliribble commented 2026-05-26 21:09:30 +00:00
Author
Owner
Copy link

ee82100a0a deals with platform/arcgis.go

ee82100a0a8adab34b26221b14589f16b2b892cf deals with `platform/arcgis.go`
eliribble commented 2026-05-26 22:54:02 +00:00
Author
Owner
Copy link

a4b2031461 deals with platform/compliance.go

a4b2031461a4885512526a1ea534d35ce44bad56 deals with `platform/compliance.go`
eliribble referenced this issue from a commit 2026-05-30 23:20:10 +00:00
Remove platform/parcel.go
eliribble referenced this issue from a commit 2026-05-30 23:22:11 +00:00
Fix naming change
eliribble referenced this issue from a commit 2026-05-30 23:28:47 +00:00
Update vendor hash
eliribble commented 2026-06-02 15:44:05 +00:00
Author
Owner
Copy link

Fixed in 782a49f220

Fixed in 782a49f220d0e0a17c7cd7d7c56a56837b6b13c8
eliribble referenced this issue from a commit 2026-08-29 22:17:58 +00:00
Remove platform/parcel.go
eliribble referenced this issue from a commit 2026-08-29 22:17:58 +00:00
Fix naming change
eliribble referenced this issue from a commit 2026-08-29 22:17:58 +00:00
Update vendor hash
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
fix/signin-stale-impersonation
issue-164-overview-map
worried-pug
add-ts-ui-tests-action
implement-maintenance-mode
investigate-planet-mosaics-new-area-sync
upbeat-turtle-2
quiet-peacock
light-rat-2
light-rat
upbeat-turtle
reveal-ui-bundle
smart-baboon
snowy-gopher
bitter-moth
famous-mantis
evil-donkey-continued
elegant-mule
exciting-sheep
sleek-fireant
magical-armadillo
aquatic-monkey
issue-176-n-items
issue-176-reveal-ui-redesign
issue-184-nearmap-tiles
issue-183-crop-data
issue-13-train-data-api
issue-180-skip-trace-design-doc
issue-174-reveal-ui-mockup
issue-166-communication-search
issue-165-address-reporter-list-view
issue-160-h3cell-bad-data
issue-155-feature-observation-parser-v2
issue-155-feature-observation-parser
issue-152-land-use-soil-data
fix/issue-134-ci-ssh-auth
fix/golint-ci-submodule-auth
fix/golangci-lint-unstable
fix/incoming-text-missing-destination-phone
fix/csv-import-missing-srid-4326
79
78
77
76
75
74
73
72
70
50
48
46
45
_backup/pre-rewrite-tag44
44
42
41
40
39
38
37
35
34
33
v0.0.5
v0.0.4
32
31
30
29
28
27
26
25
24
v0.0.23
v0.0.22
v0.0.21
v0.0.20
v0.0.19
0.0.18
0.0.17
0.0.16
0.0.15
0.0.14
0.0.13
0.0.12
0.0.11
0.0.10
0.0.9
0.0.8
0.0.7
v0.0.6
v0.0.3
v0.0.2
v0.0.1
Labels
Clear labels
  
customer
Customers are aware of, and affected by the issue

  
prod
Issue has manifested in production

  
repro'd
Has been reproduced by engineering

  
retest
Issue is ready for retesting in staging

  
staging
Issue has manifested in staging

No labels
customer
prod
repro'd
retest
staging
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Gleipnir/nidus-sync#32
No description provided.