Haha, it probably shouldn’t! I’ll have a look.
C. On 9 Aug 2021, at 19:43, Reuven Lax <[email protected]> wrote: I didn't even realize that the proto compiler allowed camel-case field names! I think the only option would be for someone to add support for camel-case proto field names in Beam. On Mon, Aug 9, 2021 at 10:57 AM Chris Hinds <[email protected]<mailto:[email protected]>> wrote: Hi, I get an IllegalArgumentException when I call a row function against a proto instance. SerializableFunction myRowFunction = new ProtoMessageSchema().toRowFunction(new TypeDescriptor<MyDataModel.ProtoPayload>() {}); MyDataModel.ProtoPayload payload = … Row row = (Row) myRowFunction.apply(payload); It looks like there is some field name mapping down in ProtoByteBuddyUtils.getProtoGetter() and protoGetterName(...) which assumes proto field names are snake_case, when it tries to map row schema names into proto getter names. I appreciate proto best practice is to have snake_case field names. Sadly, my proto field names are lowerCamel, and the above lookup fails. Short of renaming every Proto definition, is there anything I can do? Cheers, Chris.
