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

지도 목록 메모리 초과 관련 논의 #42

Open
DPS0340 opened this issue Jun 12, 2022 · 1 comment
Open

지도 목록 메모리 초과 관련 논의 #42

DPS0340 opened this issue Jun 12, 2022 · 1 comment
Assignees
Labels
implementation Implement something to make (Component, API Integration, etc...)

Comments

@DPS0340
Copy link
Contributor

DPS0340 commented Jun 12, 2022

  • 프론트엔드 지도 목록 페이지 메모리 초과 관련 논의
    • 현재 CI/CD, 로컬 빌드 모두 메모리 초과 에러가 뜨는 상황

    • 폴리곤 렌더링(1)도 문제지만, 근본적으로 동급 데이터까지 전부 다 클로저로 변수 저장(2)되는 것이 문제라고 추측됨

      // 일단 의심되는 구간 (pages/maps/index.tsx -- useEffect())
      const polygons: Array<PolygonMetadata<KakaoPolygonType>> = [];
              const citiesPolygons: Array<PolygonMetadata<KakaoPolygonType>> = [];
              const districtPolygons: Array<PolygonMetadata<KakaoPolygonType>> = [];
      • 크롬 메모리 구조
    • 다만 최적화가 가능하지만 구현 난이도가 너무 높아 (코드베이스 자체가 Dirty) 차라리 지도를 포기하고 다중 테이블로 선택하는 것이 어떨까 생각중

      • 광역시 & 도 → 시 & 구 → 군 & 동 차례대로 선택
    • 다만 조회 가능한 행정구역만 보여줘야 한다면, 차라리 군 & 동 단위로 정렬해서 Flat하게 보여주는 것이 제일 나을수도 있음

      1. 지도 구역 일단은 전부 볼 수 있다면 10-12페이지
      2. 조회 가능한 곳만 한다면 13페이지
        • 현재 보고 있는 곳만 궁금할테니 조회 가능한 구역만 보여주면 될 것 같다.
@DPS0340
Copy link
Contributor Author

DPS0340 commented Jul 2, 2022

임시 조치로 관련 코드 삭제 조치함.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
implementation Implement something to make (Component, API Integration, etc...)
Projects
Development

No branches or pull requests

2 participants