Skip to content

Commit c1dd804

Browse files
authored
Expected exception when incorrectly configuring upsert has changed (#1878)
AbstractMROldApiSaveTest.testUpdateWithoutId broke when #1839 was merged because we are now failing earlier if upsert is used but es.mapping.id is not set. The exception is now the same as the one you get when update is not configured to use es.mapping.id. Relates #1839 #69
1 parent 624fd44 commit c1dd804

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

mr/src/itest/java/org/elasticsearch/hadoop/integration/mr/AbstractMROldApiSaveTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -287,7 +287,7 @@ public void testSaveWithIngest() throws Exception {
287287
}
288288

289289

290-
@Test(expected = IOException.class)
290+
@Test(expected = EsHadoopIllegalArgumentException.class)
291291
public void testUpdateWithoutId() throws Exception {
292292
JobConf conf = createJobConf();
293293
conf.set(ConfigurationOptions.ES_WRITE_OPERATION, "upsert");

0 commit comments

Comments
 (0)