Skip to content

Update benchmark specification#56

Merged
tzschmidt merged 28 commits intomasterfrom
update-benchmark-specification
Feb 4, 2026
Merged

Update benchmark specification#56
tzschmidt merged 28 commits intomasterfrom
update-benchmark-specification

Conversation

@tzschmidt
Copy link
Collaborator

  • Add new way to specify benchmark instances using spec.xml files
  • Fix issues with latest pyarrow version
  • Several minor fixes and improvements

Copy link
Member

@rkaminsk rkaminsk left a comment

Choose a reason for hiding this comment

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

Just one small nitpick. I would not use sys.exit but throw an exception.

@tzschmidt tzschmidt requested a review from rkaminsk February 4, 2026 10:12
Copy link
Member

@rkaminsk rkaminsk left a comment

Choose a reason for hiding this comment

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

Just one more nitpick in the docs left that should be addressed. If @BenKaufmann has no further comments, I think thins can be merged afterward.

@tzschmidt tzschmidt requested a review from rkaminsk February 4, 2026 12:49
@tzschmidt tzschmidt merged commit 4d7bc4b into master Feb 4, 2026
1 check passed
@rkaminsk rkaminsk deleted the update-benchmark-specification branch February 4, 2026 15:58
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.

3 participants