indeedeng / indeedeng/iwf

Remove `__IwfSystem_WorkerUrl` from memo

Open Beginner friendly
#516 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
661
Forks
63
PR merge metrics
No merged PRs in 30d

Description

https://github.com/indeedeng/iwf/blob/1bed2dc780aa430c590e3755002b84bc9b3583d0/service/api/service.go#L77

as https://github.com/indeedeng/iwf/pull/509 has removed the memo optimization for RPC

also `UseMemoForDataAttributesKey` can be removed in https://github.com/indeedeng/iwf/blob/1bed2dc780aa430c590e3755002b84bc9b3583d0/service/api/service.go#L116

Contributor guide

Open the contributing guide

Research direction

Start in service/api/service.go around lines 77 and 116, and read PR 509 for the related memo change. Remove the __IwfSystem_WorkerUrl and UseMemoForDataAttributesKey references, then run the Go tests or build to confirm the service still compiles and passes its checks.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
api, backend
Issue type
Refactor
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
68/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.