Skip to content

Benchmark/for join - #226

Draft
stevelordbq wants to merge 3 commits into
GoogleCloudDataproc:mainfrom
Bit-Quill:benchmark/for-join
Draft

stevelordbq wants to merge 3 commits into
GoogleCloudDataproc:mainfrom
Bit-Quill:benchmark/for-join

Conversation

@stevelordbq

Copy link
Copy Markdown
Contributor

No description provided.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the benchmarking suite to support multiple Spark and Scala versions, refactors the TPC-H schema with primary keys and interleaving, and introduces configuration options for predicate pushdown and output comparison. The reviewer feedback highlights a type mismatch for enablePredicateSql and an unused compareOutput property in the read benchmark runner, a broken file conversion script (convert.sh) that attempts to modify non-existent CSV files and omits nation.tbl, and a minor typo in the README.

Comment thread benchmark/tpch/tables/convert.sh Outdated
Comment thread benchmark/README.md Outdated
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.

1 participant