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

update method setSource(byte[] source) #26561

Merged
merged 2 commits into from
Sep 11, 2017

Conversation

jiangzhuolin
Copy link
Contributor

@jiangzhuolin jiangzhuolin commented Sep 9, 2017

…pe xContentType)

In ElasticSearch 5.5,the method setSource(byte[]) from the type IndexRequestBuilder is deprecated,use the method setSource(byte[] source,XContentType xContentType).

…pe xContentType)

In ElasticSearch 5.5,the method setSource(byte[]) from the type IndexRequestBuilder is deprecated,use the method setSource(byte[] source,XContentType xContentType).
@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@jiangzhuolin jiangzhuolin changed the title update setSource(byte[] source) to setSource(byte[] source,XContentTy… update method setSource(byte[] source) Sep 9, 2017
@@ -140,7 +140,7 @@ String json = "{" +
"}";

IndexResponse response = client.prepareIndex("twitter", "tweet")
.setSource(json)
      .setSource(json,XContentType.JSON)
Copy link
Member

Choose a reason for hiding this comment

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

nit: Could you add a whitespace afther the comma for better readability?

Copy link
Contributor Author

@jiangzhuolin jiangzhuolin Sep 10, 2017

Choose a reason for hiding this comment

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

hi, @cbuescher I'm so sorry about my doc style,this is my first commit for elasticsearch. and shoud I create a new pull request again?

Copy link
Member

@cbuescher cbuescher left a comment

Choose a reason for hiding this comment

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

@jiangzhuolin good catch, thanks for opening the PR. I left a tiny note about formatting, will merge the PR once you were able to change this.

add a whitespace after the comma for better readability.
@cbuescher
Copy link
Member

@elasticmachine test this please

@cbuescher cbuescher merged commit e96cb56 into elastic:5.5 Sep 11, 2017
cbuescher pushed a commit that referenced this pull request Sep 11, 2017
The method setSource(byte[]) from the type IndexRequestBuilder is deprecated, use the method setSource(byte[] source, XContentType xContentType).

* add a whitespace after the comma

add a whitespace after the comma for better readability.
cbuescher pushed a commit that referenced this pull request Sep 11, 2017
The method setSource(byte[]) from the type IndexRequestBuilder is deprecated, use the method setSource(byte[] source, XContentType xContentType).

* add a whitespace after the comma

add a whitespace after the comma for better readability.
cbuescher pushed a commit that referenced this pull request Sep 11, 2017
The method setSource(byte[]) from the type IndexRequestBuilder is deprecated, use the method setSource(byte[] source, XContentType xContentType).

* add a whitespace after the comma

add a whitespace after the comma for better readability.
cbuescher pushed a commit that referenced this pull request Sep 11, 2017
The method setSource(byte[]) from the type IndexRequestBuilder is deprecated,
use the method setSource(byte[] source, XContentType xContentType).
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Sep 12, 2017
…rflow

* origin/master: (59 commits)
  Fix Lucene version of 5.6.1.
  Remove azure deprecated settings (elastic#26099)
  Handle the 5.6.0 release
  Allow plugins to validate cluster-state on join (elastic#26595)
  Remove index mapper dynamic settings (elastic#25734)
  update AWS SDK for ECS Task IAM support in discovery-ec2 (elastic#26479)
  Azure repository: Accelerate the listing of files (used in delete snapshot) (elastic#25710)
  Build: Remove norelease from forbidden patterns (elastic#26592)
  Fix reference to painless inside expression engine (elastic#26528)
  Build: Move javadoc linking to root build.gradle (elastic#26529)
  Test: Remove leftover static bwc test case (elastic#26584)
  Docs: Remove remaining references to file and native scripts (elastic#26580)
  Snapshot fallback should consider build.snapshot
  elastic#26496: Set the correct bwc version after backport to 6.x
  Fix the MapperFieldType.rangeQuery API. (elastic#26552)
  Deduplicate `_field_names`. (elastic#26550)
  [Docs] Update method setSource(byte[] source) (elastic#26561)
  [Docs] Fix typo in javadocs (elastic#26556)
  Allow multiple digits in Vagrant 2.x minor versions
  Support Vagrant 2.x
  ...
@lcawl lcawl removed the v6.1.0 label Dec 12, 2017
@jimczi jimczi added v7.0.0-beta1 and removed v7.0.0 labels Feb 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants