Ankit Kala

Results 59 comments of Ankit Kala

Hi @skumarp7, There is no one dedicatedly supporting the repository (unless its a critical release blocker). Feel free to debug the issue. If there are any ideas/hypothesis you've that you...

Also tagging @saikaranam-amazon for review

I've some concerns around moving the classes out of cross cluster replication package. Ideally either of these 2 options should solve the problem: Option 1: Move the ISM action into...

> @ankitkala I tried the method of using ISM spi, but seems still have the same problem > > If we extend ReplicationPlugin with ISM spi, that means CCR can...

> have to be recreated/reloaded into ccr's classloader Hey @aggarwalShivani, what i don't understand is why we need to load the ccr classes at runtime. Compile time dependency makes sense,...

> > @ankitkala can your team help our contributor here to enable this workflow you are thinking? I'm not sure if CCR is publishing any jar for now. > >...

> Hi @ankitkala Any suggestions here? > > > One **issue** I'm facing is -> [Common-utils](https://github.com/opensearch-project/common-utils/pull/744/files) and [ISM](https://github.com/opensearch-project/index-management/pull/1276) projects have now updated to the **base-line jdk to 21** from 11....

Not sure if its related to this change: https://github.com/apache/lucene/commit/c8e05c8cd6e018133d3d739d159d5b3b6b29c179#diff-be8459728adce212557bc19afbe0a8b91c788a49b1379323af40e3ba3ec4e479R154 Can you retry after disabling this feature by adding this JVM arg: `"-Dorg.apache.lucene.store.MMapDirectory.enableMemorySegments=false"`

Might be related to https://github.com/opensearch-project/OpenSearch/pull/17502 We can do similar change here: https://github.com/opensearch-project/cross-cluster-replication/blob/main/src/main/kotlin/org/opensearch/replication/repository/RestoreContext.kt#L42

Were you able to mitigate this by using jvm argument shared above? I'll check if this can be fixed for upcoming versions.