Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add test coverage for Painless general casting of boolean and Boolean #37780

Merged
merged 2 commits into from
Jan 30, 2019

Conversation

jdconrad
Copy link
Contributor

This adds test coverage for general casts in Painless between boolean and other types and Boolean and other types.

@jdconrad jdconrad added >test Issues or PRs that are addressing/adding tests :Core/Infra/Scripting Scripting abstractions, Painless, and Mustache v7.0.0 v6.7.0 labels Jan 23, 2019
@jdconrad jdconrad requested a review from rjernst January 23, 2019 16:34
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@jdconrad
Copy link
Contributor Author

@elasticmachine test this please

1 similar comment
@jdconrad
Copy link
Contributor Author

@elasticmachine test this please

@jdconrad
Copy link
Contributor Author

@elasticmachine run elasticsearch-ci/1

1 similar comment
@jdconrad
Copy link
Contributor Author

@elasticmachine run elasticsearch-ci/1

Copy link
Member

@rjernst rjernst left a comment

Choose a reason for hiding this comment

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

LGTM

@jdconrad
Copy link
Contributor Author

@elasticmachine run elasticsearch-ci/1

2 similar comments
@jdconrad
Copy link
Contributor Author

@elasticmachine run elasticsearch-ci/1

@jdconrad
Copy link
Contributor Author

@elasticmachine run elasticsearch-ci/1

@jdconrad jdconrad merged commit 36ee78d into elastic:master Jan 30, 2019
jdconrad added a commit that referenced this pull request Jan 30, 2019
…#37780)

This adds test coverage for general casts in Painless between boolean and other types and Boolean and other types.
@jdconrad
Copy link
Contributor Author

@rjernst Thanks for the review!

jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Jan 30, 2019
* master:
  Remove types from watcher docs (elastic#38002)
  Add test coverage for Painless general casting of boolean and Boolean (elastic#37780)
  Fixed test bug, lastFollowTime is null if there are no follower indices.
  Add ECS schema for user-agent ingest processor (elastic#37727) (elastic#37984)
  Extract TransportRequestDeduplication from ShardStateAction (elastic#37870)
  Expose retention leases in shard stats (elastic#37991)
jasontedor added a commit to dnhatn/elasticsearch that referenced this pull request Jan 31, 2019
* elastic/master:
  ILM setPriority corrections for a 0 value (elastic#38001)
  Temporarily disable BWC for retention lease stats (elastic#38049)
  Skip Shrink when numberOfShards not changed (elastic#37953)
  Add dispatching to `HandledTransportAction` (elastic#38050)
  Update httpclient for JDK 11 TLS engine (elastic#37994)
  Reduce flaxiness of ccr recovery timeouts test (elastic#38035)
  Fix ILM status to allow unknown fields (elastic#38043)
  Fix ILM Lifecycle Policy to allow unknown fields (elastic#38041)
  Update verify repository to allow unknown fields (elastic#37619)
  [ML] Datafeed deprecation checks (elastic#38026)
  Deprecate minimum_master_nodes (elastic#37868)
  Remove types from watcher docs (elastic#38002)
  Add test coverage for Painless general casting of boolean and Boolean (elastic#37780)
  Fixed test bug, lastFollowTime is null if there are no follower indices.
  Add ECS schema for user-agent ingest processor (elastic#37727) (elastic#37984)
  Extract TransportRequestDeduplication from ShardStateAction (elastic#37870)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/Scripting Scripting abstractions, Painless, and Mustache >test Issues or PRs that are addressing/adding tests v6.7.0 v7.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants