Alb3e3 commented on code in PR #51027:
URL: https://github.com/apache/arrow/pull/51027#discussion_r3896974078
##########
dev/archery/archery/integration/datagen.py:
##########
@@ -2047,6 +2078,8 @@ def get_generated_json_files(tempdir=None):
# TODO(https://github.com/apache/arrow/issues/38045)
.skip_format(SKIP_FLIGHT, '.NET')
.skip_tester('Ruby'),
+
+ generate_extension_wrapped_union_case(),
Review Comment:
Confirmed. After removing the bypass, there were still two C++ integration
consumers missing extension-type registration: `flight-test-integration-client`
and `c_data_integration_internal`.
With just those registrations reverted locally, `archery integration
--run-flight --run-c-data -x -k extension_union` fails in Flight with
`Extension type 'sparse-union-extension' not found` while opening
`generated_extension_union.json`.
I pushed f215787 to register the dense/sparse union extension types in both
places, and rerunning the same focused `extension_union` Flight + C Data
command passes locally.
--
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]
