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
Fold engine into IndexShard #9181
Conversation
@@ -23,6 +23,7 @@ | |||
import org.elasticsearch.common.logging.Loggers; | |||
import org.elasticsearch.common.settings.Settings; | |||
import org.elasticsearch.index.settings.IndexSettings; | |||
import org.elasticsearch.indices.cluster.IndicesClusterStateService; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Extra unneeded additions in this file?
LGTM, this is a nice simplification! |
private long gcDeletesInMillis = DEFAULT_GC_DELETES.millis(); | ||
|
||
private volatile boolean enableGcDeletes = true; | ||
private volatile String codecName = DEFAUTL_CODEC; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DEFAUTL -> DEFAULT
Left some comments, but I like the simplification a lot! |
I iterated here a bit and added all kinds of cleanups on the engine. I also added documentation to the new EngineConfig and removed unused methods. I think it's ready, unless anybody objects I will push soon |
LGTM |
This commit removes most of the Engine abstractions and removes Engine exposure via dependency injection. It also removes the Holder abstraction and makes the engine itself start at constrcution time. It removes the start method from the engine entire which means no engine instances exists until it's started. There is also no way to stop the engine to restart, it needs to be an entire new Engine
I pushed this to 1.x as well |
This commit removes most of the Engine abstractions and removes
Engine exposure via dependency injection. It also removes the Holder
abstraction and makes the engine itself start at constrcution time.
It removes the start method from the engine entire which means no engine
instances exists until it's started. There is also no way to stop the
engine to restart, it needs to be an entire new Engine