[Java][FlightSQL] Inconsistent GET_PRIMARY_KEYS_SCHEMA

[devinrsmith]

Describe the bug, including details regarding any error messages, version, and platform.

The protobuf documentation ¹ for the returned schema of FlightSQL CommandGetPrimaryKeys is

 *  catalog_name: utf8,
 *  db_schema_name: utf8,
 *  table_name: utf8 not null,
 *  column_name: utf8 not null,
 *  key_name: utf8,
 *  key_sequence: int32 not null

org.apache.arrow.flight.sql.FlightSqlProducer.Schemas#GET_PRIMARY_KEYS_SCHEMA ² is defined as

    public static final Schema GET_PRIMARY_KEYS_SCHEMA =
        new Schema(asList(
            Field.nullable("catalog_name", VARCHAR.getType()),
            Field.nullable("db_schema_name", VARCHAR.getType()),
            Field.notNullable("table_name", VARCHAR.getType()),
            Field.notNullable("column_name", VARCHAR.getType()),
            Field.notNullable("key_sequence", INT.getType()),
            Field.nullable("key_name", VARCHAR.getType())));

Note the reordering of key_name and key_sequence.

Should this be considered a bug in the implementation or the documentation? Or, is field order "unimportant" in this context?

Component(s)

Java

Footnotes

1. https://github.com/apache/arrow/blob/apache-arrow-17.0.0/format/FlightSql.proto#L1280-L1285 ↩

2. https://github.com/apache/arrow/blob/apache-arrow-17.0.0/java/flight/flight-sql/src/main/java/org/apache/arrow/flight/sql/FlightSqlProducer.java#L1164-L1172 ↩

[lidavidm]

C++ agrees with Java:

https://github.com/apache/arrow/blob/2bbd67dc42c64bb5185b6b9147b777d9230e76e9/cpp/src/arrow/flight/sql/server.cc#L1302-L1308

So does Go:

https://github.com/apache/arrow-go/blob/56b794f52a9bfb45fb54a4e070002a69b38fb018/arrow/flight/flightsql/schema_ref/reference_schemas.go#L46-L53

I'd consider this a bug in the documentation.