Skip to content

Commit

Permalink
fix(config): remove spark config defaults
Browse files Browse the repository at this point in the history
  • Loading branch information
ebuildy committed Nov 16, 2022
1 parent 5b32bde commit 325cb78
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 52 deletions.
1 change: 0 additions & 1 deletion dev/default.conf
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,6 @@ djobi {
driver {
allowMultipleContexts = false
memory = "256M"
host = "localhost"
}

es {
Expand Down
28 changes: 1 addition & 27 deletions djobi-cli/src/main/release/defaults.conf
Original file line number Diff line number Diff line change
Expand Up @@ -76,40 +76,14 @@ djobi {

executors {
spark {
master = "local"

webHistoryUrlForJob = "http://localhost:4040"

data {

}

conf {
spark {
driver {
allowMultipleContexts = false
memory = "256M"
host = "localhost"
}

es {
scroll.size = 100
nodes.wan.only= true
mapping.date.rich = false
net {
#ssl = true
ssl.cert.allow.self.signed = true
}
}

sql {
shuffle.partitions = 10
}

eventLog.enabled = false
ui.enabled = false
}
}
}
}
}
}
23 changes: 0 additions & 23 deletions djobi-core/src/main/resources/reference.conf
Original file line number Diff line number Diff line change
Expand Up @@ -82,30 +82,7 @@ djobi {
}

conf {
spark {
driver {
allowMultipleContexts = false
memory = "256M"
host = "localhost"
}

es {
scroll.size = 100
nodes.wan.only= true
mapping.date.rich = false
net {
#ssl = true
ssl.cert.allow.self.signed = true
}
}

sql {
shuffle.partitions = 10
}

eventLog.enabled = false
ui.enabled = false
}
}
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ void shouldConfigureSparkContext() throws IOException {

final SparkContext context = executor.getSparkContext();

Assertions.assertEquals("djobi-app", context.appName());
Assertions.assertEquals("djobi", context.appName());

Assertions.assertEquals("true", context.getConf().get("spark.es.nodes.wan.only"));
}
Expand Down

0 comments on commit 325cb78

Please sign in to comment.