Re: [PR] [fix](regression) Stabilize external table guardrail cases [doris]
Gabriel39 merged PR #66509: URL: https://github.com/apache/doris/pull/66509 -- 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]
Re: [PR] [fix](regression) Stabilize external table guardrail cases [doris]
Gabriel39 commented on PR #66509: URL: https://github.com/apache/doris/pull/66509#issuecomment-5205542733 run buildall -- 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]
Re: [PR] [fix](regression) Stabilize external table guardrail cases [doris]
yiguolei commented on PR #66509: URL: https://github.com/apache/doris/pull/66509#issuecomment-5203645644 run buildall -- 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]
Re: [PR] [fix](regression) Stabilize external table guardrail cases [doris]
github-actions[bot] commented on PR #66509: URL: https://github.com/apache/doris/pull/66509#issuecomment-5199558803 PR approved by anyone and no changes requested. -- 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]
Re: [PR] [fix](regression) Stabilize external table guardrail cases [doris]
github-actions[bot] commented on PR #66509: URL: https://github.com/apache/doris/pull/66509#issuecomment-5199558564 PR approved by at least one committer and no changes requested. -- 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]
Re: [PR] [fix](regression) Stabilize external table guardrail cases [doris]
github-actions[bot] commented on PR #66509: URL: https://github.com/apache/doris/pull/66509#issuecomment-5199462956 Codex automated review failed and did not complete. Error: All Codex review accounts are usage-limited; earliest retry is 2026-08-08T03:32:00Z. Workflow run: https://github.com/apache/doris/actions/runs/31063690460 Please trigger /review again after that time. -- 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]
Re: [PR] [fix](regression) Stabilize external table guardrail cases [doris]
Gabriel39 commented on PR #66509: URL: https://github.com/apache/doris/pull/66509#issuecomment-5199453072 run buildall -- 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]
Re: [PR] [fix](regression) Stabilize external table guardrail cases [doris]
Gabriel39 commented on PR #66509: URL: https://github.com/apache/doris/pull/66509#issuecomment-5199453270 /review -- 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]
Re: [PR] [fix](regression) Stabilize external table guardrail cases [doris]
github-actions[bot] commented on PR #66509: URL: https://github.com/apache/doris/pull/66509#issuecomment-5194497585 Codex automated review failed and did not complete. Error: All Codex review accounts are usage-limited; earliest retry is 2026-08-08T03:32:00Z. Workflow run: https://github.com/apache/doris/actions/runs/31023047435 Please trigger /review again after that time. -- 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]
Re: [PR] [fix](regression) Stabilize external table guardrail cases [doris]
Gabriel39 commented on PR #66509: URL: https://github.com/apache/doris/pull/66509#issuecomment-5194141202 /review -- 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]
Re: [PR] [fix](regression) Stabilize external table guardrail cases [doris]
Gabriel39 commented on PR #66509: URL: https://github.com/apache/doris/pull/66509#issuecomment-5194140098 run buildall -- 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]
Re: [PR] [fix](regression) Stabilize external table guardrail cases [doris]
hello-stephen commented on PR #66509: URL: https://github.com/apache/doris/pull/66509#issuecomment-5194140125 Thank you for your contribution to Apache Doris. Don't know what should be done next? See [How to process your PR](https://cwiki.apache.org/confluence/display/DORIS/How+to+process+your+PR). Please clearly describe your PR: 1. What problem was fixed (it's best to include specific error reporting information). How it was fixed. 2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be. 3. What features were added. Why was this function added? 4. Which code was refactored and why was this part of the code refactored? 5. Which functions were optimized and what is the difference before and after the optimization? -- 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]
[PR] [fix](regression) Stabilize external table guardrail cases [doris]
Gabriel39 opened a new pull request, #66509: URL: https://github.com/apache/doris/pull/66509 ### What problem does this PR solve? The external-table regression suite had two environment-dependent failures: - The Paimon guardrail case attempted to create a table with an empty field name. Paimon rejects empty and whitespace-only field names while constructing the table schema, before Doris can scan it. - The Iceberg Variant case required more than one scanner to report rows. File scanner tasks pull ranges dynamically, so minimum scanner concurrency does not guarantee that every scheduled scanner consumes rows from a small multi-file fixture. ### What is changed? - Remove the unsupported empty-field Paimon fixture and its expected results. Delimiter-rich field names remain covered end to end, while empty-segment framing remains covered by the codec unit test. - Keep the parallel Iceberg Variant correctness query, but remove the assertion on nondeterministic per-scanner range assignment. ### Validation - `test_paimon_jni_reader_guardrails`: passed - `test_iceberg_variant_read`: passed twice consecutively ### Release note None. These are test-only corrections. -- 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]
