Oleks V

regression: `BigDecimal::from_str` gives unexpected result

7

Please find a small test case which reflects the concern ``` #[tokio::test] async fn testx() -> Result { use std::str::FromStr; let bd = BigDecimal::from_str(&10f64.to_string()).unwrap().normalized().to_string(); assert_eq!(bd, "10"); Ok(()) } ``` in...

Document Sort Merge Join algorithm

4

### Is your feature request related to a problem or challenge? I faced that it is challenging to understand fully how Sort Merge Join `sort_merge_join.rs` implemented and would be nice...

Fix: Sort Merge Join LeftSemi issues when JoinFilter is set

2

## Which issue does this PR close? Closes #10379 . ## Rationale for this change Fixing some existing SMJ LeftSemi bugs when join filter is set. Currently the join either...

[DISCUSSION] Sort Merge Join Experimental status

12

### Is your feature request related to a problem or challenge? Hi all I was going through SMJ implementation and suddenly stepped on the comments ``` // Sort-Merge join support...

### Describe the bug The ticket is umbrella for Sort Merge Join LeftSemi bugs. Currently Sort Merge Join LeftSemi crashes or gives the wrong result. ### To Reproduce To reproduce...

Sort Merge Join. LeftAnti issues

1

### Describe the bug SMJ LeftAnti join returns the wrong result ### To Reproduce ``` #[tokio::test] async fn test_anti() -> Result { let ctx: SessionContext = SessionContext::new(); let sql =...

Minor: Generate the supported Spark builtin expression list into MD file

1

## Which issue does this PR close? Follow up on #331 Closes #282 Related #240 ## Rationale for this change Generate the supported Spark builtin expression list into MD file...

feat: Comet windows functions support

1

## Which issue does this PR close? Closes #22. ## Rationale for this change ## What changes are included in this PR? ## How are these changes tested?

feat: Make spark builtin expression coverage report more readable

1

I'm wondering if we can produce a markdown document instead of plain text. So it can be displayed in format. We can do it in follow up. _Originally posted by...

Would you mind to upload the coverage result as an artifact so that every CI run could produce a test report? The related part is at https://github.com/apache/arrow-datafusion-comet/commit/56ccf6a3cc1471ce4445be21a7fe21911e8fd33c#diff-02adba3bf9683d1c5c48e061e0108e79aea4f6d9cfbafebfd935fd3470fb7140R64 _Originally posted by...