map = new HashMap() 초기화는 문제를 유발하나요?

Question

증상톰캣을 1주일 정도 가동하면 톰캣 프로세스(java)가 10초마다 주기로 반복해서 CPU 100% 올라가고 서비스가 느려짐10초 주기는 아마도 가비지 컬랙터 추측조치ChatHandler.chatEnterCntMap = new HashMap();→ ChatHandler.chatEnterCntMap.clear(); 로 변경하니 문제가 해결된 것 같음new HashMap()으로 초기화 해도 아무 문제가 없을 것 같은데문제가 있는 코드일까요?// 웹소켓 핸들러 @Component public class ChatHandler extends TextWebSocketHandler { // 채팅방 입장 횟수 public static Map chatEnterCntMap = new HashMap(); // 채팅방 입장 처리 private void ENTER(WebSocketSession session, ChatMsgDto chatMsgDto) throws Exception { // 채팅방 입장시 ip별로 횟수 +1 증가 synchronized (ChatHandler.class) { if (chatEnterCntMap.get(ip) == null) { chatEnterCntMap.put(ip, 1); } else { Integer cnt = chatEnterCntMap.get(ip) + 1; chatEnterCntMap.put(ip, cnt); } } } } // 스케쥴러 @Component public class Scheduler { // 90초마다 실행 @Scheduled(fixedDelay = 90000) public void cronJob5() throws Exception { // 초기화 synchronized (ChatHandler.class) { ChatHandler.chatEnterCntMap = new HashMap(); } } }

yeori · Accepted Answer

'여기가 문제겠군' 생각하고 조치를 취했으나 문제지점이 아닐 때가 많습니다.CPU 100%의 원인이 진짜 gc 때문인지 확인을 해야 합니다. 모든 조치는 그 다음입니다.

카스마당 · Answer

synchronized 제거하고 전용 동기화맵 사용하는 게 좋아 보이는데..public static final ConcurrentHashMap chatEnterCntMap = new ConcurrentHashMap<>();그러고 ChatHandler.chatEnterCntMap.clear(); 처리 등

캐티 · Answer

java concurrent counter 이거를 검색해보면 록을 치는 방법이 있는 반면에 원자 단위로 치는거 있어요. 물론 싱글 스레드를 사용하면 성능이 낮은 서버에서도 록을 치지 아니하여도 대빈담.위에 거는 일반적으로 문제가 나지 않아요. GC가 돌기 이전까지 메모리가 밀리는 현상땜 서버가 멈추는 거는 심하겟쩌. 일반적으로 clear 사용 하고요. 본래 의심이 된다 이럼 내부 코드를 봐바요. 캐패시터를 가지고 있는데 배열로 null을 가지고 있스빈담.jdk/src/java.base/share/classes/java/util/HashMap.java at master · openjdk/jdk · GitHub위에서 문제는 웹 소켓 연결이 종료되었는데 close를 치지 않아서 난리가 났을 거여요. 웹소켓 인스턴스를 종료하지 않았다 이거나 ..

kenu,전독시 · Answer

몇 명이 사용하는 지요.

월급은나의빛 · Answer

new 로 "초기화"를 하면 기존 object 가 gc를 통해 회수된다는 걸 믿는 건데, 그렇지 않을 수 있습니다.또한 clear 와 비교해 성능이나 메모리 사용면에서 효율적이지 않습니다. 기존 object 를 버리고 새로 만들어야 하는 명확한 이유가 없으면 clear를 사용하세요.

퍼대기 · Answer

10일째 되니 다시 똑같은 증상이 나타나서 톰캣 재기동 했습니다.이게 아닌가 보네요.

map = new HashMap() 초기화는 문제를 유발하나요?

6개의 답변

Q&A 목록