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

섹션 데이터베이스 sqlite3로 변경 #92

Merged
merged 5 commits into from
Dec 29, 2024

Conversation

onee-only
Copy link
Member

  • 섹션 데이터베이스를 lmdb에서 sqlite3로 변경
  • 비동기 처리를 위해 sqlite3 내장 라이브러리 대신 aiosqlite 라이브러리 사용
  • SectionStorage 인터페이스를 async로 변경 -> 사용코드도 async로 변경

발생할 수 있는 문제:
BoardHandler가 비동기적으로 처리됨에 따라 보드 관련 리시버들은 원자적이지 않게 되었고, 따라서 섹션에 대한 레이스 컨디션이 생기게 되었습니다.

레이스 컨디션이 발생해서 생기는 문제는 타일 데이터의 일관성이 무너지거나, 같은 타일에 대한 인터랙션이 중복으로 생길 수 있다는 것 정도가 있습니다. 제 판단으론 이 문제들은 베타테스트 전까지 필요한 기능을 개발하는 것보다 우선순위가 떨어진다고 생각하여 건드리지 않았습니다.

@onee-only onee-only requested a review from byundojin December 28, 2024 15:18
@onee-only onee-only force-pushed the enhancement/section-database-sqlite3 branch from 8867fda to 4628db2 Compare December 28, 2024 23:09
@onee-only onee-only mentioned this pull request Dec 29, 2024
@onee-only onee-only merged commit 039b92a into develop Dec 29, 2024
1 check passed
@onee-only onee-only deleted the enhancement/section-database-sqlite3 branch December 29, 2024 13:29
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.

2 participants