Skip to content

Updating Postgres/Postgis Call and Postgis to 3.0 #6528

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
Merged
merged 69 commits into from
Mar 23, 2020
Merged

Updating Postgres/Postgis Call and Postgis to 3.0 #6528

merged 69 commits into from
Mar 23, 2020

Conversation

@cbaker6
Copy link
Contributor

@cbaker6 cbaker6 commented Mar 23, 2020 •
edited
Loading

Close #6441

cbaker6 added 30 commits March 13, 2020 23:34
@cbaker6
Attempting to fix Postgres issue
841768c
@cbaker6
Attempting to fix Postgres issue
33da65e 
trying to stop loop
@cbaker6
Attempting to fix Postgres
43b714d 
isolating postgres calls
@cbaker6
Attempting to fix Postgres issue
8010633 
Separating jobs
@cbaker6
Attempting to fix postgres
cc2506a
@cbaker6
Attempting to fix postgres
05181a7
@cbaker6
Attempting to fix postgres
1ff769d 
Separating builds again
@cbaker6
Attempting to fix postgres
fba1b83
@cbaker6
Attempting to fix postgres
edf60e8
@cbaker6
Attempting to fix postgres
c884e5d 
Just added back version 10, just in case it gets called
@cbaker6
Attempting to fix postgres
e553200
@cbaker6
Attempting to fix postgres
c7b4411
@cbaker6
Attempting to fix postgres
063dd2f
@cbaker6
Attempting to fix postgres
d357a47
@cbaker6
Attempting to fix postgres
b048966
@cbaker6
Attempting to fix postgres
299b91d
@cbaker6
Attempting to fix postgres
6adf026
@cbaker6
Attempting to fix postgres
87e37a1
@cbaker6
Attempting to fix postgres
c20a82e
@cbaker6
Attempting to fix postgres
76a29d0
@cbaker6
Update .travis.yml
b90abc4
@cbaker6
Attempting to fix postgres
5d4019d 
Removed postgres installs from unneeded test cases. Added the ability to test Postgres 10 and 11
@cbaker6
Attempting to fix postgres
028daee
@cbaker6
Attempting to fix postgres
387258c
@cbaker6
Attempting to fix postgres
dc3afa6
@cbaker6
Attempting to fix postgres
06045a5 
Added test for postgres 12 that's allowed to fail
@cbaker6
Attempting to fix postgres
ebcbbb0
@cbaker6
Attempting to fix postgres
72f72a1 
Second round to see if it fails eventually
@cbaker6
Attempting to fix postgres
2047e2e 
Round 3
@cbaker6
Attempting to fix postgres
fa10637 
Allowing all postgres to fail since it seems to occur randomly
cbaker6 added 24 commits March 17, 2020 10:06
@cbaker6
Merge branch 'master' of https://github.com/netreconlab/parse-server
ac3be67
@cbaker6
Update .travis.yml
0052963
@cbaker6
add sourcing in script
a375604
@cbaker6
trying to fix source
8d26cbc
@cbaker6
fixing env var in script
dfcbe0d
@cbaker6
fixed ; near then
a5aa417
@cbaker6
Cleaning up travis file
cc91dc4 
removed old lines
@cbaker6
Finishing clean up
0e33072
@cbaker6
Fixing allow_failures since "name" was removed
773ca67
@cbaker6
Update .travis.yml
fe0077c
@cbaker6
Removed Postgres 11 from allow_failures
559e0a7
@cbaker6
I think using travis default postgres port of 5433 will allow us to n…
2ceff98 
…ot have to remove anything from the image
@cbaker6
Merge branch 'master' of https://github.com/netreconlab/parse-server
57a81cb
@cbaker6
Switching travis to postgres port 5433
7776457
@cbaker6
modifying script for test
59a3e8a
@cbaker6
modifying script for test
3a099c1
@cbaker6
modifying script for test
fc2006d
@cbaker6
reverting back to working way with removing postgres from image
4ffee37
@cbaker6
Reverted back to removing postgres from image
8d17b04
@cbaker6
removing postgres 12
84b4a64
@cbaker6
removed postgres-12 from allow_failures
839d3e6
@cbaker6
updated postgres method from deprecated. Also updating postgis to 3.0
c1173cb
@cbaker6
updated postgis to 3.0
648ea74
@cbaker6
Merge branch 'master' into master
3842083
@cbaker6
Copy link
Contributor Author

cbaker6 commented Mar 23, 2020 •
edited
Loading

Fix #6441. Added updated documentation in #6506

@codecov
Copy link

codecov bot commented Mar 23, 2020 •
edited
Loading

Codecov Report

Merging #6528 into master will increase coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6528      +/-   ##
==========================================
+ Coverage   93.98%   94.01%   +0.03%     
==========================================
  Files         169      169              
  Lines       11828    11828              
==========================================
+ Hits        11116    11120       +4     
+ Misses        712      708       -4
Impacted Files Coverage Δ
...dapters/Storage/Postgres/PostgresStorageAdapter.js 96.74% <ø> (ø) ⬆️
src/RestWrite.js 93.81% <0%> (+0.16%) ⬆️
src/Adapters/Storage/Mongo/MongoStorageAdapter.js 93.46% <0%> (+0.67%) ⬆️

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 b60502d...6a6c2d4. Read the comment docs.

@cbaker6
Copy link
Contributor Author

cbaker6 commented Mar 23, 2020

@dplewis this one is ready for your review

@dplewis dplewis merged commit 927676b into parse-community:master Mar 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@dplewis dplewis dplewis approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

PostgresStorageAdapter.js: postgis version aware use of distance sphere function (GeoPoint distance calculation)

2 participants

@cbaker6 @dplewis