Feature/new arena service poc #6702
Conversation
add battle action
jonny-jeahyunchoi
requested review from
sonohoshi and
eugene-doobu
as code owners
|
@jonny-jeahyunchoi The versions of bundled Libplanet assemblies and Libplanet submodule vendored by Lib9c apparently do not match:
Leave a comment in this pull_request with the following command to let the bot upgrade bundled Libplanet assemblies:
|
| var description = response.Value["description"]?.ToString() ?? ""; | ||
| // 설명에서 유효한 파라미터 이름 생성 | ||
| var descriptionPart = Regex.Replace(description, @"[^a-zA-Z0-9]", ""); | ||
| if (descriptionPart.Length > 30) |
There was a problem hiding this comment.
특별한이유는 없습니다 이름이 너무길어지면 보기에 불편할까봐 잘랐어요
| } | ||
| } | ||
|
|
||
| public async Task<SeasonResponse> GetSeasonByBlockAsync(Int64 blockIndex) |
There was a problem hiding this comment.
반환 형태를 unitask로 안하신 이유가 있을까요?
There was a problem hiding this comment.
특별한이유없습니다~! 변경해도 상관없을것같네요 이후 작업에서 변경해보겠습니다~!
| @@ -100,6 +104,12 @@ public ActionManager(IAgent agent) | |||
| foreach (var gameAction in gameActions) | |||
| { | |||
| _actionIdToTxIdBridge[gameAction.Id] = (tx.Id, _agent.BlockIndex); | |||
| var existingAction = _cachedPostProcessedActions.FirstOrDefault(a => ReferenceEquals(a.Item1, gameAction)); | |||
There was a problem hiding this comment.
이 용도면 queue쓰는게 더 좋을 것 같습니다. 봐야 알거같긴 한데 스레드 문제도 생길 수 있어보이네요
There was a problem hiding this comment.
이건 서명후 비동기처리하는 작업부분에서 수정되었습니다~! #6718
| var existingAction = _cachedPostProcessedActions.FirstOrDefault(a => ReferenceEquals(a.Item1, gameAction)); | ||
| if (existingAction != null) | ||
| { | ||
| existingAction.Item2?.Invoke(tx.Id); |
There was a problem hiding this comment.
액션을 처리하는 시점에 액션 실행이 목표같긴 한데 first or deafult로 처리하는 이유를 잘 모르겠습니다
There was a problem hiding this comment.
이건 임시로 서명id를가지고 서비스요청을 보내야해서 사용한거고 케시된게 하나밖에없을거라는 가정하에 작업된 임시코드입니다 서명후 비동기처리하는 작업부분에서 수정되었습니다~! #6718
fix action pipeline
아레나 서비스 POC 구현