Skip to content

[Backport 2.19-dev] Fix span on negative timestamp#4028

Merged
LantaoJin merged 1 commit into2.19-devfrom
backport/backport-4017-to-2.19-dev
Aug 19, 2025
Merged

[Backport 2.19-dev] Fix span on negative timestamp#4028
LantaoJin merged 1 commit into2.19-devfrom
backport/backport-4017-to-2.19-dev

Conversation

@opensearch-trigger-bot
Copy link
Copy Markdown
Contributor

Backport ad3fc1f from #4017.

* Fix span on negative timestamp

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Fix span on negative timestamp

Signed-off-by: Heng Qian <qianheng@amazon.com>

* typo

Signed-off-by: Heng Qian <qianheng@amazon.com>

* Refine code

Signed-off-by: Heng Qian <qianheng@amazon.com>

---------

Signed-off-by: Heng Qian <qianheng@amazon.com>
(cherry picked from commit ad3fc1f)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@dai-chen
Copy link
Copy Markdown
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
Copy Markdown
Collaborator

@qianheng-aws could you check the CI failure?

@qianheng-aws
Copy link
Copy Markdown
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
Copy Markdown
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