Skip to content

fix(sql): fix nanosecond issues in ASOF join TOLERANCE and CSV imports#6183

Merged
bluestreak01 merged 2 commits intomasterfrom
victor_nano_fix
Sep 25, 2025
Merged

fix(sql): fix nanosecond issues in ASOF join TOLERANCE and CSV imports#6183
bluestreak01 merged 2 commits intomasterfrom
victor_nano_fix

Conversation

@kafka1991
Copy link
Copy Markdown
Collaborator

@kafka1991 kafka1991 commented Sep 25, 2025

Fixed two bugs on nanos:

  • ASOF Join TOLERANCE nano interval bug: Fixed incorrect nanosecond precision handling when calculating TOLERANCE values for ASOF joins.

  • CSV Import timestamp type detection: Fixed automatic timestamp type selection during CSV imports when timestamp columns are explicitly typed.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Sep 25, 2025

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch victor_nano_fix

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@kafka1991 kafka1991 changed the title fix(sql): fix a few nano's issue fix(sql): fix nanosecond issues in ASOF joins and CSV imports Sep 25, 2025
@kafka1991 kafka1991 changed the title fix(sql): fix nanosecond issues in ASOF joins and CSV imports fix(sql): fix nanosecond issues in ASOF joins TOLERANCE and CSV imports Sep 25, 2025
@kafka1991 kafka1991 changed the title fix(sql): fix nanosecond issues in ASOF joins TOLERANCE and CSV imports fix(sql): fix nanosecond issues in ASOF join TOLERANCE and CSV imports Sep 25, 2025
@glasstiger
Copy link
Copy Markdown
Contributor

[PR Coverage check]

😍 pass : 3 / 3 (100.00%)

file detail

path covered line new line coverage
🔵 io/questdb/griffin/SqlCodeGenerator.java 2 2 100.00%
🔵 io/questdb/cutlass/text/TextMetadataParser.java 1 1 100.00%

@bluestreak01 bluestreak01 merged commit 146824a into master Sep 25, 2025
33 of 35 checks passed
@bluestreak01 bluestreak01 deleted the victor_nano_fix branch September 25, 2025 12:28
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