Skip to content

Conversation

@opensearch-trigger-bot
Copy link
Contributor

Backport ad3fc1f from #4017.

* Fix span on negative timestamp

Signed-off-by: Heng Qian <[email protected]>

* Fix span on negative timestamp

Signed-off-by: Heng Qian <[email protected]>

* typo

Signed-off-by: Heng Qian <[email protected]>

* Refine code

Signed-off-by: Heng Qian <[email protected]>

---------

Signed-off-by: Heng Qian <[email protected]>
(cherry picked from commit ad3fc1f)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@dai-chen
Copy link
Collaborator

2025-08-13T03:35:50.5364120Z 5093 tests completed, 1 failed, 150 skipped
2025-08-13T03:35:50.5365330Z Tests with failures:
2025-08-13T03:35:50.6363650Z  - org.opensearch.sql.calcite.tpch.CalcitePPLTpchIT.testQ7

I've seen this failure before. Is it flaky?

@noCharger
Copy link
Collaborator

@qianheng-aws could you check the CI failure?

@qianheng-aws
Copy link
Collaborator

qianheng-aws commented Aug 15, 2025

I've seen this failure before. Is it flaky?

org.opensearch.sql.calcite.tpch.CalcitePPLTpchIT > testQ7 FAILED
    java.net.SocketTimeoutException: 60,000 milliseconds timeout on connection http-outgoing-470 [ACTIVE]

It's flakey and not related to this change. I've seen it several times before.

Shall we disable q7 until we find the root cause of the flakey? @LantaoJin

@LantaoJin
Copy link
Member

I've seen this failure before. Is it flaky?

org.opensearch.sql.calcite.tpch.CalcitePPLTpchIT > testQ7 FAILED
    java.net.SocketTimeoutException: 60,000 milliseconds timeout on connection http-outgoing-470 [ACTIVE]

It's flakey and not related to this change. I've seen it several times before.

Shall we disable q7 until we find the root cause of the flakey? @LantaoJin

Okey, let me add some retry for q7. The q7 may fail sometimes with socket timeout on mac/windows.

@LantaoJin LantaoJin merged commit 6c4a966 into 2.19-dev Aug 19, 2025
69 of 76 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants