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

issue #120 관련. yg-amazon-exp branch 정리를 위한 cherry-picking #121

Merged
merged 1 commit into from
Jan 9, 2021

Conversation

ygnkim
Copy link
Member

@ygnkim ygnkim commented Jan 7, 2021

#120 에 나온것과 같이, stale branch인 yg-amazon-exp를 정리하는 과정에서, 하나의 commit만을 히스토리로 남기기위해 master로 cherry-picking하였다.

해당 커밋은 BLEEP lib로 PoW node를 만들어 아마존에서 동작시킬때, 예기치 않은 오류를 방지하기 위한 fix이다.

@victork86 victork86 self-requested a review January 8, 2021 01:38
@victork86
Copy link
Collaborator

victork86 commented Jan 8, 2021

EstimateRTTModule에서 데이터 소켓을 얻기전, 소켓 연결상태를 점검하는 로직이 추가되었네요.
CI결과 등 확인하였고 별다른 이상 없어 보입니다. 수고하셨습니다. 그리고 Merge 권한 부탁드립니다.^^

@ygnkim ygnkim merged commit 9ae9712 into master Jan 9, 2021
@ygnkim
Copy link
Member Author

ygnkim commented Jan 9, 2021

일단 이 PR을 master에 병합하는거니까 제가 병합했습니다. 다음부터는 develop 브랜치에 우선적으로 병합하도록 하면 권한 있으실 겁니다~

@ygnkim ygnkim deleted the issue/120 branch January 9, 2021 03:38
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