Addressed comment on PR

Signed-off-by: Shubh Sahu <[email protected]>

CHANGELOG.md

@@ -17,13 +17,13 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 - Add cluster primary balance contraint for rebalancing with buffer ([#12656](https://github.com/opensearch-project/OpenSearch/pull/12656))
 - [Remote Store] Make translog transfer timeout configurable ([#12704](https://github.com/opensearch-project/OpenSearch/pull/12704))
 - Reject Resize index requests (i.e, split, shrink and clone), While DocRep to SegRep migration is in progress.([#12686](https://github.com/opensearch-project/OpenSearch/pull/12686))
-- [Remote Store] Add capability of doing refresh as determined by the translog ([#12992](https://github.com/opensearch-project/OpenSearch/pull/12992))
 - Add support for more than one protocol for transport ([#12967](https://github.com/opensearch-project/OpenSearch/pull/12967))
 - [Tiered Caching] Add dimension-based stats to ICache implementations. ([#12531](https://github.com/opensearch-project/OpenSearch/pull/12531))
 - Add changes for overriding remote store and replication settings during snapshot restore. ([#11868](https://github.com/opensearch-project/OpenSearch/pull/11868))
 - Add an individual setting of rate limiter for segment replication ([#12959](https://github.com/opensearch-project/OpenSearch/pull/12959))
 - [Streaming Indexing] Ensure support of the new transport by security plugin ([#13174](https://github.com/opensearch-project/OpenSearch/pull/13174))
 - Add cluster setting to dynamically configure the buckets for filter rewrite optimization. ([#13179](https://github.com/opensearch-project/OpenSearch/pull/13179))
+- [Remote Store] Add capability of doing refresh as determined by the translog ([#12992](https://github.com/opensearch-project/OpenSearch/pull/12992))
 
 ### Dependencies
 - Bump `org.apache.commons:commons-configuration2` from 2.10.0 to 2.10.1 ([#12896](https://github.com/opensearch-project/OpenSearch/pull/12896))

server/src/main/java/org/opensearch/index/translog/RemoteFsTranslog.java

@@ -683,7 +683,7 @@ int availablePermits() {
      * @return {@code true} if the shard should be flushed
      */
     @Override
-    public boolean shouldFlush() {
+    protected boolean shouldFlush() {
         return readers.size() >= translogTransferManager.getMaxRemoteTranslogReadersSettings();
     }
 }

server/src/main/java/org/opensearch/index/translog/Translog.java

@@ -2088,7 +2088,7 @@ public long getMinUnreferencedSeqNoInSegments(long minUnrefCheckpointInLastCommi
      * each translog type can have it's own decider
      * @return {@code true} if the shard should be flushed
      */
-    public boolean shouldFlush() {
+    protected boolean shouldFlush() {
         return false;
     }
 }