Skip to content

Conversation

@filipecosta90
Copy link
Collaborator

@filipecosta90 filipecosta90 commented May 8, 2020

@codecov
Copy link

codecov bot commented May 8, 2020

Codecov Report

Merging #70 into master will increase coverage by 0.22%.
The diff coverage is 84.21%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #70      +/-   ##
==========================================
+ Coverage   75.04%   75.27%   +0.22%     
==========================================
  Files          12       12              
  Lines        1182     1197      +15     
==========================================
+ Hits          887      901      +14     
- Misses        231      233       +2     
+ Partials       64       63       -1     
Impacted Files Coverage Δ
redisearch/schema.go 86.71% <84.21%> (-0.79%) ⬇️
redisearch/client.go 80.15% <0.00%> (+0.52%) ⬆️

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 51a0cb3...55aa74e. Read the comment docs.

@gkorland gkorland requested review from itamarhaber and removed request for gkorland May 10, 2020 20:21
// To enable the temporary index creation, use SetTemporaryPeriod(). This method should be preferably used for disabling the flag
func (options *Options) SetTemporary(temporary bool) *Options {
options.Temporary = temporary
return options

Choose a reason for hiding this comment

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

Not sure someone will create a temporary object and then regret ... but it's ok to have this api I guess..

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I believe it makes sense if you want to reuse options across several creations. WDYT?

@@ -0,0 +1,26 @@
# Go parameters
GOCMD=go

Choose a reason for hiding this comment

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

Is it common on go project to have a Makefile?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I knew someone would pick on the GOCMD =)
regarding the makefile I believe it helps both for developing, manual testing and CI. cc @rafie

Copy link

@MeirShpilraien MeirShpilraien left a comment

Choose a reason for hiding this comment

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

LGTM, added small comments but its not a blocker...

@filipecosta90 filipecosta90 merged commit 69202df into master May 11, 2020
@filipecosta90 filipecosta90 deleted the temporary.indices branch May 11, 2020 09:10
@filipecosta90 filipecosta90 linked an issue Jul 9, 2020 that may be closed by this pull request
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.

support for [TEMPORARY {seconds}] on FT.CREATE

3 participants