Re: [PR] Minor: Interval singleton [datafusion]

2025-04-26 Thread via GitHub
comphead merged PR #15859: URL: https://github.com/apache/datafusion/pull/15859 -- 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: github-unsubscr...@dataf

Re: [PR] Minor: Interval singleton [datafusion]

2025-04-25 Thread via GitHub
jayzhan211 commented on code in PR #15859: URL: https://github.com/apache/datafusion/pull/15859#discussion_r2061085496 ## datafusion/expr-common/src/interval_arithmetic.rs: ## @@ -286,6 +286,11 @@ impl Interval { } } +/// Create a new `Interval` with the same

Re: [PR] Minor: Interval singleton [datafusion]

2025-04-25 Thread via GitHub
m09526 commented on code in PR #15859: URL: https://github.com/apache/datafusion/pull/15859#discussion_r2060372212 ## datafusion/expr-common/src/interval_arithmetic.rs: ## @@ -286,6 +286,11 @@ impl Interval { } } +/// Create a new `Interval` with the same low

Re: [PR] Minor: Interval singleton [datafusion]

2025-04-25 Thread via GitHub
m09526 commented on code in PR #15859: URL: https://github.com/apache/datafusion/pull/15859#discussion_r2060371205 ## datafusion/expr-common/src/interval_arithmetic.rs: ## @@ -286,6 +286,11 @@ impl Interval { } } +/// Create a new `Interval` with the same low

[PR] Minor: Interval singleton [datafusion]

2025-04-25 Thread via GitHub
jayzhan211 opened a new pull request, #15859: URL: https://github.com/apache/datafusion/pull/15859 ## Which issue does this PR close? - Closes #. ## Rationale for this change We don't need additional check for the same value. ## What changes are inc