Refactor benchmark jobs and remove hardcoded paths (Fixes #418)

[makarandhinge]

Description

This PR revisits and cleans up benchmark jobs in the wayang-benchmark module as requested in issue #418.

Based on maintainer feedback, the original JavaNativeAPI examples were preserved and equivalent implementations using JavaPlanBuilder were added instead of replacing them. This keeps both APIs available for benchmarking and comparison.

Changes Made

Benchmark Improvements

• Fixed hardcoded file paths in Grep.java.

Added JavaPlanBuilder Implementations

• Added TPCHQ1WithPlanBuilder.java as a JavaPlanBuilder version of TPCHQ1WithJavaNative.java.
• Added WordCountWithPlanBuilder.java as a JavaPlanBuilder version of WordCountWithJavaNativeAPI.java.

Code Cleanup

• Cleaned up Main.java by removing unused imports and adding Configuration.
• Cleaned up WordCountParquet.java by removing unused imports and adding Configuration.
• Cleaned up WordCount.java by removing commented code.

Current Benchmark Structure

wayang-benchmark/src/main/java/org/apache/wayang/apps/
├── grep/
│   └── Grep.java
├── tpch/
│   ├── TPCHQ1WithJavaNative.java
│   └── TPCHQ1WithPlanBuilder.java
└── wordcount/
    ├── Main.java
    ├── WordCount.java
    ├── WordCountParquet.java
    ├── WordCountWithJavaNativeAPI.java
    └── WordCountWithPlanBuilder.java

Testing

• Verified that all benchmark jobs compile successfully.
• Ran benchmark examples locally to ensure functionality remains unchanged.

Related Issue

Fixes #418

[zkaoudi]

Thanks @makarandhinge. It would be better if we keep the JavaNativeAPI examples and add the respective ones with the planbuilder instead of replacing them.

[makarandhinge]

Thank you for the suggestion @zkaoudi.

I will restore the original JavaNativeAPI examples and add separate implementations using JavaPlanBuilder instead of replacing them. This way both approaches will remain available in the benchmark module.

I will update the PR accordingly.

[makarandhinge]

Hi @zkaoudi,

I have updated the PR accordingly. The original JavaNativeAPI examples are now preserved, and separate implementations using JavaPlanBuilder have been added.

Please let me know if any further adjustments are needed. Thanks!

[zkaoudi]

Thank you @makarandhinge

[zkaoudi]

What's the purpose of adding an empty configuration in the context?

[zkaoudi]

same here

[zkaoudi]

There is already a Wordcount with the planbuilder so we do not need a second benchmark.

See here: https://github.com/apache/wayang/blob/main/wayang-benchmark/src/main/java/org/apache/wayang/apps/wordcount/WordCount.java

[makarandhinge]

Thanks, @zkaoudi
I’ve made the requested changes, you can proceed. Let me know if anything else needs adjustment.