Gabriel39 commented on PR #68667:
URL: https://github.com/apache/doris/pull/68667#issuecomment-5909496335
Fixed the new P0 failure in `test_flight_cancel_cleanup` in ae5067f6a8c.
Flight tickets format query ID halves without leading zeroes, while
`/api/query_pipeline_tasks/{query_id}` requires exactly two 16-digit
hexadecimal halves. A shorter high half caused HTTP 500 before the cleanup
assertion could run. The test now pads each half before calling the diagnostic
endpoint and keeps the cancellation assertions unchanged.
Validation: the original ticket-decoding block failed focused tests for
short ID halves. The updated block passes short/full-width IDs and canonical
deduplication. Against a real local BE, a short ID reproduced HTTP 500, while
normalized IDs returned HTTP 200 with the expected missing-query response.
Groovy compilation passed, and the temporary BE was stopped. Full
cancellation-suite validation remains for CI.
--
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]
