Skip to content

Commit

Permalink
Fixed a flaky test in `S3DataSegmentPusherConfigTest#testSerializatio…
Browse files Browse the repository at this point in the history
…n` by changing string to key:value pair (apache#15207)

* Fix capacity response in mm-less ingestion (apache#14888)

Changes:
- Fix capacity response in mm-less ingestion.
- Add field usedClusterCapacity to the GET /totalWorkerCapacity response.
This API should be used to get the total ingestion capacity on the overlord.
- Remove method `isK8sTaskRunner` from interface `TaskRunner`

* Using Map to perform comparison

* Minor Change

---------

Co-authored-by: George Shiqi Wu <george.wu@imply.io>
  • Loading branch information
2 people authored and ycp2 committed Nov 17, 2023
1 parent 669cb3e commit 48d3828
Showing 1 changed file with 3 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,9 @@ public void testSerialization() throws IOException
+ "\"disableAcl\":false,\"maxListingLength\":2000,\"useS3aSchema\":false}";

S3DataSegmentPusherConfig config = JSON_MAPPER.readValue(jsonConfig, S3DataSegmentPusherConfig.class);
Assert.assertEquals(jsonConfig, JSON_MAPPER.writeValueAsString(config));
Map<String, String> expected = JSON_MAPPER.readValue(jsonConfig, Map.class);
Map<String, String> actual = JSON_MAPPER.readValue(JSON_MAPPER.writeValueAsString(config), Map.class);
Assert.assertEquals(expected, actual);
}

@Test
Expand Down

0 comments on commit 48d3828

Please sign in to comment.