Gabriel39 opened a new pull request, #68770:
URL: https://github.com/apache/doris/pull/68770
### What problem does this PR solve?
Flight SQL GetSqlInfo always reports server version `1.0` and Arrow version
`18.2.0`, so ADBC clients receive stale metadata after upgrades. Report the
running FE build version and short commit hash, using the same format as FE
heartbeat metadata, and read the Arrow version from the loaded Flight SQL JAR.
If the Arrow manifest has no implementation version, report `unknown`. Keep
the info code registered because Arrow's SqlInfoBuilder rejects explicit
requests for unregistered codes.
### Release note
Fix stale server and Arrow library versions returned by Flight SQL
GetSqlInfo.
### Validation
- Added three Flight RPC tests covering all SQL info, the server version
alone, and the Arrow version alone.
- All three tests reproduce the stale version bug before the fix and pass
after it.
- All three also pass with the Flight SQL dependency loaded from an exploded
directory without its manifest; Arrow allocator debug checks are enabled.
- Compiled the changed producer and tests against the existing FE dependency
classpath. No full FE/BE build or deployed-cluster test was run.
- FE reactor Checkstyle validation passed with zero violations.
### Check List (For Author)
- [x] Unit Test
- Behavior changed:
- [x] Yes. Version metadata follows the running server and Arrow library.
- Does this need documentation?
- [x] No.
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
