[CALCITE-5558] avatica protobuf update_count and other signed ints should not be unsigned

• Type: Bug
• Status: Open
• Priority: Minor
• Resolution: Unresolved
• Affects Version/s: avatica-1.23.0
• Component/s: avatica
• Assignee: Unassigned
• Reporter: Martin Jonsson
• Created: 06/Mar/23 10:56
• Updated: 07/Mar/23 18:54

Description

the update_count field of result set response in avatica is defined as uint64 which by default takes negative value -1.

The protobuf specification states that negative values should use int or sint, not uint, since uint is... unsigned. For this reason many protobuf implementations can't handle uints that are negative. This might not be a terrible if you are building a client but I'm building a server based on avatica protobuf protocol and I would like it to work not just with standard java implementation and c++ implementation which seems to be the ones that handles this odd case.

Would it be possible to change update_count from uint64 to int64? and same for all other possible negative ints currently defined as unsigned?

 

Many thanks

Martin