Skip to content

Commit cbaf61a

Browse files
authored
Merge pull request #492 from DataDog/gary/muzzle-sparkjava
Muzzle and Expand Spark Java Instrumentation
2 parents 7ecc49a + 33ab8d0 commit cbaf61a

File tree

5 files changed

+10
-11
lines changed

5 files changed

+10
-11
lines changed

dd-java-agent/instrumentation/sparkjava-2.4/sparkjava-2.4.gradle renamed to dd-java-agent/instrumentation/sparkjava-2.3/sparkjava-2.3.gradle

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,11 @@
1-
apply plugin: 'version-scan'
2-
3-
versionScan {
4-
group = "com.sparkjava"
5-
module = 'spark-core'
6-
versions = "[2.4,)"
7-
verifyPresent = [
8-
"spark.route.Routes": null
9-
]
1+
// building against 2.3 and testing against 2.4 because JettyHandler is available since 2.4 only
2+
muzzle {
3+
pass {
4+
group = "com.sparkjava"
5+
module = 'spark-core'
6+
versions = "[2.3,)"
7+
assertInverse = true
8+
}
109
}
1110

1211
apply from: "${rootDir}/gradle/java.gradle"
@@ -37,7 +36,7 @@ compileJava {
3736
}
3837

3938
dependencies {
40-
compileOnly group: 'com.sparkjava', name: 'spark-core', version: '2.4'
39+
compileOnly group: 'com.sparkjava', name: 'spark-core', version: '2.3'
4140

4241
compile project(':dd-java-agent:agent-tooling')
4342
compile deps.bytebuddy

dd-java-agent/instrumentation/sparkjava-2.4/src/main/java/datadog/trace/instrumentation/sparkjava/RoutesInstrumentation.java renamed to dd-java-agent/instrumentation/sparkjava-2.3/src/main/java/datadog/trace/instrumentation/sparkjava/RoutesInstrumentation.java

File renamed without changes.

dd-java-agent/instrumentation/sparkjava-2.4/src/test/groovy/SparkJavaBasedTest.groovy renamed to dd-java-agent/instrumentation/sparkjava-2.3/src/test/groovy/SparkJavaBasedTest.groovy

File renamed without changes.

dd-java-agent/instrumentation/sparkjava-2.4/src/test/java/TestSparkJavaApplication.java renamed to dd-java-agent/instrumentation/sparkjava-2.3/src/test/java/TestSparkJavaApplication.java

File renamed without changes.

settings.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ include ':dd-java-agent:instrumentation:play-2.4'
4949
include ':dd-java-agent:instrumentation:ratpack-1.4'
5050
include ':dd-java-agent:instrumentation:servlet-2'
5151
include ':dd-java-agent:instrumentation:servlet-3'
52-
include ':dd-java-agent:instrumentation:sparkjava-2.4'
52+
include ':dd-java-agent:instrumentation:sparkjava-2.3'
5353
include ':dd-java-agent:instrumentation:spring-web'
5454
include ':dd-java-agent:instrumentation:spring-webflux'
5555
include ':dd-java-agent:instrumentation:spymemcached-2.12'

0 commit comments

Comments
 (0)