[THRIFT-5396] deprecate "Async" method postfix

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.15.0
• Component/s: netstd - Compiler
• Labels: breaking_change
• Patch Info: Patch Available
• Assignee: Jens Geyer
• Reporter: Jens Geyer
• Created: 09/Apr/21 21:23
• Updated: 11/Sep/21 12:30
• Resolved: 10/Apr/21 08:14

Description

The nestdt compiler generates method names witnh "Async" following an older recommendation for async methods in NET. It also helped to distinguish between synchronous and async/await calls in the old "csharp" bindings.

Because with modern NET versions common mainstream (and technically also the only possible way left) to call Thrift methods is now the async style, this postfix has become obsolete and starts to become a burden rather a feature.

Hence, default should be to leave method mames undecorated and instead make the "Async" postfix a compatibility option that must be enabled explicitly.

 

Issue Links

• links to: GitHub Pull Request #2374