-
Notifications
You must be signed in to change notification settings - Fork 0
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
🔨 [Chore] PostGIS 설정 #17
Conversation
- PostgreSQL 이미지를 PostGIS 확장을 포함한 이미지로 변경 - 새로운 DB 생성시 PostGIS를 활성화하는 sql을 실행 - member, restaurant에 location 컬럼 추가
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아... 위키 읽고 왔는데 플랫폼 문제가 있었군요... docker hub 가서 확인해보니 지원 플랫폼에 amd64만 지원한다고 써져있네요..
로제타 덕분에 돌아가기는 할 것 같은데 썩 좋은 방법은 아닌 것 같습니다. 다은님이 저번 프로젝트 하실 땐 이거 어떻게 해결하셨을까요? @ooheunda 혹시 docker 설정하신 분이 인텔맥이었으면 그대로 하셨을 수도..
저희는 큰 기능을 쓰는 게 아니니 ... 해결 방법은 두 가지가 있을 것 같습니다.
- 다른 사용자가 만들어놓은 arm64 이미지 사용하기
- 저희가 Dockerfile 만들어서 쓰기
- 저희는 딱 postgis 만 필요해서 그냥 plugin깔고 use extension 하는 sql까지 실행하는 이미지 만들어도 됩니다.
지금init.sql
만들어서 volume에 bind 하시는데 그것보다는 Dockerfile을 만들어 image 생성 단계에서 실행하는 게 좋은 방법이라고 생각합니다.init.sql
은 처음 세팅할 때만 필요하고 날아가는 파일이니까 volume 매핑까진 필요없을 것 같아요!
@jis-kim 저희 프로젝트 할 땐 관련된 이슈는 따로 없었습니다 ㄱ-... 도커 무지렁이라 리뷰하며 이거저거 찾아보고 있었는데 정말 생각치도 못한 문제군여... |
- 플랫폼 이슈로 multi-platform 지원하는 PostgreSQL 이미지에서 Dockerfile 생성 - 최소 설정만 넣은 Dockerfile 입니다.
- .env.sample 참고해서 .env의 host 를 맞게 작성하셔야 작동합니다.
도커파일 작성완료 했습니다.. volume 삭제했다가 다시 |
@jis-kim 수고하셨습니다~ |
- location 컬럼에서 위경도 데이터를 조회할 수 있어 lon, lat 컬럼을 따로 두지 않아도 됨
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
짱 멋있다... 고생하셨습니다~!!👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM~
Summary
Describe your changes
docker-entrypoint-initdb.d
폴더에init.sql
에 쿼리를 추가했습니다.member
,restaurant
테이블에geometry
타입의 location 컬럼 추가하였습니다.Issue number and link
closed #14
closed #18