Skip to content

Conversation

@hendrikmuhs
Copy link

improve test to run overflow forecast a 2nd time as regression test for elastic/ml-cpp#110

Note: This should right now fail as long as the cpp part hasn't been merged and artifacts are available.

@hendrikmuhs hendrikmuhs added >test Issues or PRs that are addressing/adding tests review v7.0.0 :ml Machine learning v6.4.0 labels May 30, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core

Copy link
Member

@davidkyle davidkyle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hendrikmuhs
Copy link
Author

@elasticmachine retest this please

@hendrikmuhs hendrikmuhs force-pushed the fix-deletion-forecast-overflow branch 2 times, most recently from 1a07804 to 250d4da Compare June 4, 2018 11:18
@hendrikmuhs hendrikmuhs force-pushed the fix-deletion-forecast-overflow branch from 250d4da to 3c6d8c1 Compare June 4, 2018 13:03
@hendrikmuhs hendrikmuhs merged commit 5e48ba7 into elastic:master Jun 5, 2018
martijnvg added a commit that referenced this pull request Jun 5, 2018
* es/master:
  Take into account the return value of TcpTransport.readMessageLength(...) in Netty4SizeHeaderFrameDecoder
  Move caching of the size of a directory to `StoreDirectory`. (#30581)
  Clarify docs about boolean operator precedence. (#30808)
  Docs: remove notes on sparsity. (#30905)
  Fix MatchPhrasePrefixQueryBuilderTests#testPhraseOnFieldWithNoTerms
  run overflow forecast a 2nd time as regression test for elastic/ml-cpp#110 (#30969)
  Improve documentation of dynamic mappings. (#30952)
  Decouple MultiValueMode. (#31075)
  Docs: Clarify constraints on scripted similarities. (#31076)
  Update get.asciidoc (#31084)
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Jun 5, 2018
* elastic/master:
  [Tests] Muting RatedRequestsTests#testXContentParsingIsNotLenient
  TEST:  Retry synced-flush if ongoing ops on primary (elastic#30978)
  Fix docs build.
  Only auto-update license signature if all nodes ready (elastic#30859)
  Add BlobContainer.writeBlobAtomic() (elastic#30902)
  Add a doc value format to binary fields. (elastic#30860)
  Take into account the return value of TcpTransport.readMessageLength(...) in Netty4SizeHeaderFrameDecoder
  Move caching of the size of a directory to `StoreDirectory`. (elastic#30581)
  Clarify docs about boolean operator precedence. (elastic#30808)
  Docs: remove notes on sparsity. (elastic#30905)
  Fix MatchPhrasePrefixQueryBuilderTests#testPhraseOnFieldWithNoTerms
  run overflow forecast a 2nd time as regression test for elastic/ml-cpp#110 (elastic#30969)
  Improve documentation of dynamic mappings. (elastic#30952)
  Decouple MultiValueMode. (elastic#31075)
  Docs: Clarify constraints on scripted similarities. (elastic#31076)
hendrikmuhs pushed a commit that referenced this pull request Jun 6, 2018
…p#110 (#30969)

Improve test to run overflow forecast a 2nd time as regression test for elastic/ml-cpp#110
@hendrikmuhs hendrikmuhs deleted the fix-deletion-forecast-overflow branch October 18, 2018 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:ml Machine learning >test Issues or PRs that are addressing/adding tests v6.4.0 v7.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants