fix: logging and connect db refactoring#89
Conversation
ehddnr301
left a comment
There was a problem hiding this comment.
👍 조금 더 구조화된 코드인것으로 보여 너무 좋습니다! @anjaaaaeeeellll 님도 확인 한번 해주시면 좋을것 같습니다!
💬 DB selectbox를 조정할때마다 연결 로깅이 남네요! 요건 streamlit의 구조때문인것 같습니다ㅎㅎ 링크에서 When users interact with widgets, your script reruns from top to bottom, and your app's frontend is updated 라는 문장을 발견했습니다.
|
|
||
| except Exception as e: | ||
| logger.exception("An error occurred while executing SQL: %s", e) | ||
| raise |
There was a problem hiding this comment.
👍 이 부분을 처음봐서 GPT에 질문해보았는데 아래와 같다고 하네요!
- raise → 현재 예외를 그대로 위로 전파 (traceback 유지 ✅)
- raise e → 예외 객체는 같지만, traceback은 새로 시작 (원래 위치 정보 ❌)
@ehddnr301 오 저도 이걸 확인했는데, 옵션을 수정할때마다 재설정을 하는 것 같습니다..! |
anjaaaaeeeellll
left a comment
There was a problem hiding this comment.
👍 누추한 곳에 귀한분의 코드가..너무 깔끔해졌네요! 감사합니다ㅎㅎ
|
@anjaaaaeeeellll 앗.. 영광입니당!! 확인해주셔서 감사합니다~ |

#️⃣ Issue Number
📝 요약(Summary)
💬 To Reviewers (선택)
PR Checklist
reference) How to Code Review