SkipLuceneindexing flag for DynamoPlayer#14440
SkipLuceneindexing flag for DynamoPlayer#14440reddyashish wants to merge 1 commit intoDynamoDS:RC2.19.2_masterfrom
Conversation
|
|
|
I believe we need a few more changes in D4R repo - that is we need some journal key there that will passed on as skip lucene in the startup config here. That is so that we can stop the indexing from DynamoPlayer context. But it looks like we can hold off with this fix in the near future and deal with the performance. |
|
@BogdanZavu @twastvedt With the recent performance improvement, do you think if we still need this change? |
|
The numbers you posted look really good. I'd say no, at least for now. |
|
I think we can hold off on this change for now. |
|
@BogdanZavu @twastvedt Thank you for confirmation. But I think indexing is not strictly required especially in service mode. |
|
Yep, definitely agree that ideally we'd be able to turn off indexing. Maybe always in both CLI and service modes? I agree with Bogdan that we don't need a flag, just don't have it on where it's not needed. |
Purpose
This is to test Dynamo player with 2.19.2 version. It is to skip the lucene indexing for DynamoPlayer and GD.
Declarations
Check these if you believe they are true
*.resxfilesRelease Notes
SkipLuceneindexing flag for DynamoPlayer
Reviewers
@QilongTang @BogdanZavu