Skip to content

migrated to typed actor in plugin-tester-scala; it will be also used …#699

Closed
kazuf3 wants to merge 2 commits into
apache:mainfrom
kazuf3:feature/typed-actor-in-pluginTesterScala
Closed

migrated to typed actor in plugin-tester-scala; it will be also used …#699
kazuf3 wants to merge 2 commits into
apache:mainfrom
kazuf3:feature/typed-actor-in-pluginTesterScala

Conversation

@kazuf3
Copy link
Copy Markdown
Contributor

@kazuf3 kazuf3 commented May 17, 2026

…for doc: server/walkthrough/stateful services section

(See #698)

…for doc: server/walkthrough/stateful services section
@kazuf3
Copy link
Copy Markdown
Contributor Author

kazuf3 commented May 17, 2026

Do I also have to update build.gradle?

Copy link
Copy Markdown
Member

@pjfanning pjfanning left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can't we keep the legacy actor and create a new sample using the typed actor?

@pjfanning
Copy link
Copy Markdown
Member

can't we keep the legacy actor and create a new sample using the typed actor?

I would suggest leaving https://github.com/apache/pekko-grpc/tree/main/plugin-tester-scala/src/main/scala/example/myapp/helloworld as is and adding a new typedhelloworld package in the same plugin-tester-scala module that uses the typed actor.

@kazuf3
Copy link
Copy Markdown
Contributor Author

kazuf3 commented May 17, 2026

OK, ill create a new branch/PR with:

  • new class path/file names
  • change the document reference annotation

@kazuf3
Copy link
Copy Markdown
Contributor Author

kazuf3 commented May 18, 2026

Updated PR to #702. Closing this.

@kazuf3 kazuf3 closed this May 18, 2026
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