Skip to content

Commit

Permalink
[MINOR] improvement(test): Disable Spark UI for Spark integration tes…
Browse files Browse the repository at this point in the history
…ts (#1831)

### What changes were proposed in this pull request?

Disable Spark UI for Spark integration tests.

### Why are the changes needed?

Spark UI is not necessary for test cases.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Existing UTs.
  • Loading branch information
wForget authored Jun 26, 2024
1 parent 30cefcc commit f8e4329
Showing 1 changed file with 4 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,10 @@ protected Map runSparkApp(SparkConf sparkConf, String testFileName) throws Excep
}

protected SparkConf createSparkConf() {
return new SparkConf().setAppName(this.getClass().getSimpleName()).setMaster("local[4]");
return new SparkConf()
.setAppName(this.getClass().getSimpleName())
.setMaster("local[4]")
.set("spark.ui.enabled", "false");
}

public void updateSparkConfWithRssGrpc(SparkConf sparkConf) {
Expand Down

0 comments on commit f8e4329

Please sign in to comment.