Skip to content

42 Multi User환경을 위한 세션별 분리 #43

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ehddnr301
Copy link
Collaborator

#️⃣ Issue Number

📝 요약(Summary)

  • 기존에는 모든 사용자가 하나의 공유된 graph 인스턴스를 사용하여 동시 접속 시 상태 꼬임 문제가 발생할 수 있었습니다.
  • st.session_state를 활용해 세션마다 독립된 graph 인스턴스를 사용하도록 수정했습니다.

💬 To Reviewers (선택)

  • 가장 빠르게 목표를 달성할 수 있을것 같아 st.session_state를 사용했는데 적절한지 의견 부탁드립니다!

PR Checklist

  • TBD

reference) How to Code Review

  • 따봉(👍): 리뷰어가 리뷰이의 코드에서 칭찬의 의견을 남기고 싶을 때 사용합니다.
  • 느낌표(❗): 리뷰어가 리뷰이에게 필수적으로 코드 수정을 요청할 때 사용합니다.
  • 물음표 (❓): 리뷰어가 리뷰이에게 의견을 물어보고 싶을 때 사용합니다.
  • 알약 (💊): 리뷰어가 리뷰이의 코드에서 개선된 방법을 제안하지만 그것의 반영이 필수까지는 아닐 때 사용합니다.

@ehddnr301 ehddnr301 linked an issue Apr 7, 2025 that may be closed by this pull request
1 task
Copy link
Contributor

@ParkGyeongTae ParkGyeongTae left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 좋은 것 같습니다~

@nonegom
Copy link
Contributor

nonegom commented Apr 7, 2025

메모리를 활용해서 Multi user를 구분하는 것도 하나의 방법이 될 수 있지 않을까 싶습니다.

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.

Multi User환경을 위한 세션별 분리
3 participants