Skip to content

Relax assertions in segment level field stats#111243

Merged
dnhatn merged 1 commit intoelastic:mainfrom
dnhatn:relax-assertions
Jul 24, 2024
Merged

Relax assertions in segment level field stats#111243
dnhatn merged 1 commit intoelastic:mainfrom
dnhatn:relax-assertions

Conversation

@dnhatn
Copy link
Copy Markdown
Member

@dnhatn dnhatn commented Jul 24, 2024

This PR relaxes the assertions to allow an additional field introduced in serverless.

@dnhatn dnhatn added >test Issues or PRs that are addressing/adding tests :StorageEngine/Mapping The storage related side of mappings labels Jul 24, 2024
@dnhatn dnhatn requested a review from mosche July 24, 2024 15:58
@dnhatn dnhatn marked this pull request as ready for review July 24, 2024 15:59
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-storage-engine (Team:StorageEngine)

@dnhatn dnhatn force-pushed the relax-assertions branch from 622d8cb to b8100bd Compare July 24, 2024 16:00
@dnhatn dnhatn requested a review from lkts July 24, 2024 16:00
@dnhatn
Copy link
Copy Markdown
Member Author

dnhatn commented Jul 24, 2024

Thanks Sasha!

@dnhatn dnhatn merged commit 52834fe into elastic:main Jul 24, 2024
@dnhatn dnhatn deleted the relax-assertions branch July 24, 2024 19:58
weizijun added a commit to weizijun/elasticsearch that referenced this pull request Jul 25, 2024
* main: (39 commits)
  Update README.asciidoc (elastic#111244)
  ESQL: INLINESTATS (elastic#109583)
  ESQL: Document a little of `DataType` (elastic#111250)
  Relax assertions in segment level field stats (elastic#111243)
  LogsDB data generator - support nested object field (elastic#111206)
  Validate `Authorization` header in Azure test fixture (elastic#111242)
  Fixing HistoryStoreTests.testPut() and testStoreWithHideSecrets() (elastic#111246)
  [ESQL] Remove Named Expcted Types map from testing infrastructure  (elastic#111213)
  Change visibility of createWriter to allow tests from a different package to override it (elastic#111234)
  [ES|QL] Remove EsqlDataTypes (elastic#111089)
  Mute org.elasticsearch.repositories.azure.AzureBlobContainerRetriesTests testReadNonexistentBlobThrowsNoSuchFileException elastic#111233
  Abstract codec lookup by name, to make CodecService extensible (elastic#111007)
  Add HTTPS support to `AzureHttpFixture` (elastic#111228)
  Unmuting tests related to free_context action being processed in ESSingleNodeTestCase (elastic#111224)
  Upgrade Azure SDK (elastic#111225)
  Collapse transport versions for 8.14.0 (elastic#111199)
  Make sure contender uses logs templates (elastic#111183)
  unmute HistogramPercentileAggregationTests.testBoxplotHistogram (elastic#111223)
  Refactor Quality Assurance test infrastructure (elastic#111195)
  Mute org.elasticsearch.xpack.restart.FullClusterRestartIT testDisableFieldNameField {cluster=UPGRADED} elastic#111222
  ...

# Conflicts:
#	server/src/main/java/org/elasticsearch/TransportVersions.java
@mosche
Copy link
Copy Markdown
Contributor

mosche commented Jul 25, 2024

Thanks @dnhatn 🥇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:StorageEngine/Mapping The storage related side of mappings Team:StorageEngine >test Issues or PRs that are addressing/adding tests v8.16.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants