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

changes for elastic v7 #32

Merged
merged 3 commits into from
Jan 21, 2021
Merged

changes for elastic v7 #32

merged 3 commits into from
Jan 21, 2021

Conversation

nicpottier
Copy link
Collaborator

@nicpottier nicpottier commented Jan 20, 2021

Mostly just updating the library we are dependent on and tweaking a few parameters. For 8.0 we'll have to address the mapping issue.

@codecov-io
Copy link

codecov-io commented Jan 20, 2021

Codecov Report

Merging #32 (ad23b25) into master (9748da3) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #32   +/-   ##
=======================================
  Coverage   76.43%   76.43%           
=======================================
  Files           1        1           
  Lines         191      191           
=======================================
  Hits          146      146           
  Misses         25       25           
  Partials       20       20           
Impacted Files Coverage Δ
indexer.go 76.43% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9748da3...ad23b25. Read the comment docs.

@@ -84,7 +84,7 @@ func CreateNewIndex(url string, alias string) (string, error) {
}

// initialize our index
createURL := fmt.Sprintf("%s/%s", url, physicalIndex)
createURL := fmt.Sprintf("%s/%s?include_type_name=true", url, physicalIndex)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Needed for mapping to be supported.


// deletes a contact
const deleteCommand = `{ "delete" : { "_id": %d, "_type": "_doc", "_version": %d, "_version_type": "external", "_routing": %d} }`
const deleteCommand = `{ "delete" : { "_id": %d, "_type": "_doc", "version": %d, "version_type": "external", "routing": %d} }`
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Same parameters, just no underscores anymore.

Hits []struct {
Total struct {
Value int `json:"value"`
} `json:"total"`
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Now an object, this will have repercussions on how we query on the mailroom side.

@@ -1,5 +1,10 @@
cloud.google.com/go v0.26.0/go.mod h1:aQUYkXzVsufM+DwF1aE+0xfcU+56JwCaLick0ClmMTw=
Copy link
Member

Choose a reason for hiding this comment

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

looks like a little go mod tidy wouldn't go amiss

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thought the same, but no, these are all still there afterwards.

@nicpottier
Copy link
Collaborator Author

fixes #24

@nicpottier nicpottier merged commit 37b4bbb into master Jan 21, 2021
@nicpottier nicpottier deleted the elastic-v7 branch January 21, 2021 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants