[THRIFT-992] Naming convention in C# constructor is not consistent with other fields causes compile errors

• Type: Bug
• Status: Closed
• Priority: Blocker
• Resolution: Fixed
• Affects Version/s: 0.5
• Fix Version/s: 0.6
• Component/s: C# - Compiler

Environment

Using the windows binary release of thrift-0.5.0.exe on Windows 7 64-bit

• Assignee: Roger Meier
• Reporter: Nick Berardi
• Created: 05/Nov/10 12:35
• Updated: 01/Nov/11 02:51
• Resolved: 10/Nov/10 21:24

Description

While updating my FluentCassandra project using the latest Thrift-0.5.0.exe generator against the 0.7 Beta3 release of cassandra.thrift I experienced a generation problem that caused compiler errors when compiling the C# code. It appears the constructor fields are using the old naming convention for the fields, while every other part of the class seems to be using the new naming convention for fields with underscores.

You can see a diff of the files I had to manually edit here:
https://github.com/managedfusion/fluentcassandra/commit/d2d26f0bfd158cae3c39fd9cd47ec9097bc394f6

The paths starting with "FluentCassandra/Apache/Cassandra" should be the ones that you need to worry about for this bug.

Issue Links

• is related to: THRIFT-1026 C# default value code generation problem
• relates to: THRIFT-634 C# Compiler Generates Incorrect Code For Fields which begin with an uppercase letter