[Segment Replication] Update ReplicationTarget and ReplicationListener failure exception type.

[mch2]

Coming from #4278, ReplicationTarget and ReplicationListener accept OpenSearchException in their failure methods. This should be narrowed to a replication specific exception type that is used in failures from recovery / replication. Recovery already has a RecoveryFailedException, we can similarly create a ReplicationFailedException to use in the superclass.

[sreenath-tm]

I'm interested in contributing to this issue, so before I start working it, would you mind sparing your time explaining what the fix will be like and pointing me to some resources to get started.

[dreamer-89]

@sreenath-tm: Apologies for the delay on response.

The suggested fix here is to create a new exception class ReplicationFailedException (? do you have a better name). This is essentially to reflect exceptions during replication as existing OpenSearchException is a base class for all OpenSearch exception and is not accurate enough. ReplicationFailedException exception can be used in ReplicationTarget and ReplicationListener instead of OpenSearchException.

I would suggest you can start with understanding about segment replication feature using design #2229
and RFC #1694

[ayushKataria]

@dreamer-89 If no one else is working on this issue, I'll like to take it up as my first contribution to the project.

[dreamer-89]

@dreamer-89 If no one else is working on this issue, I'll like to take it up as my first contribution to the project.

Thank you @ayushKataria for the interest.
@sreenath-tm : Can you please confirm if you are still working on this issue

[dreamer-89]

@sreenath-tm: Can you please confirm if you are working on this issue ? If not, @ayushKataria would like to take this forward.

[sreenath-tm]

Hi sorry for the delay in response. @ayushKataria please feel free to take it forward.