Change all action constructors' accessor level to protected - fixes s…

…earchbox-io#215
nitram509 · Jun 28, 2015 · f4c5abc · f4c5abc
1 parent 5e272bb
commit f4c5abc
Show file tree

Hide file tree

Showing 51 changed files with 54 additions and 54 deletions.
diff --git a/jest-common/src/main/java/io/searchbox/cluster/GetSettings.java b/jest-common/src/main/java/io/searchbox/cluster/GetSettings.java
@@ -10,7 +10,7 @@
  */
 public class GetSettings extends GenericResultAbstractAction {
 
-    private GetSettings(Builder builder) {
+    protected GetSettings(Builder builder) {
         super(builder);
         setURI(buildURI());
     }

diff --git a/jest-common/src/main/java/io/searchbox/cluster/Health.java b/jest-common/src/main/java/io/searchbox/cluster/Health.java
@@ -9,7 +9,7 @@
  */
 public class Health extends GenericResultAbstractAction {
 
-    public Health(Builder builder) {
+    protected Health(Builder builder) {
         super(builder);
         setURI(buildURI());
     }

diff --git a/jest-common/src/main/java/io/searchbox/cluster/NodesHotThreads.java b/jest-common/src/main/java/io/searchbox/cluster/NodesHotThreads.java
@@ -13,7 +13,7 @@
  */
 public class NodesHotThreads extends GenericResultAbstractAction {
 
-    public NodesHotThreads(Builder builder) {
+    protected NodesHotThreads(Builder builder) {
         super(builder);
         setURI(buildURI());
     }

diff --git a/jest-common/src/main/java/io/searchbox/cluster/NodesInfo.java b/jest-common/src/main/java/io/searchbox/cluster/NodesInfo.java
@@ -9,7 +9,7 @@
  */
 public class NodesInfo extends GenericResultAbstractAction {
 
-    public NodesInfo(Builder builder) {
+    protected NodesInfo(Builder builder) {
         super(builder);
         setPathToResult("nodes");
         setURI(buildURI());

diff --git a/jest-common/src/main/java/io/searchbox/cluster/NodesShutdown.java b/jest-common/src/main/java/io/searchbox/cluster/NodesShutdown.java
@@ -9,7 +9,7 @@
  */
 public class NodesShutdown extends GenericResultAbstractAction {
 
-    public NodesShutdown(Builder builder) {
+    protected NodesShutdown(Builder builder) {
         super(builder);
         setPathToResult("nodes");
         setURI(buildURI());

diff --git a/jest-common/src/main/java/io/searchbox/cluster/NodesStats.java b/jest-common/src/main/java/io/searchbox/cluster/NodesStats.java
@@ -9,7 +9,7 @@
  */
 public class NodesStats extends GenericResultAbstractAction {
 
-    public NodesStats(Builder builder) {
+    protected NodesStats(Builder builder) {
         super(builder);
         setPathToResult("nodes");
         setURI(buildURI());

diff --git a/jest-common/src/main/java/io/searchbox/cluster/State.java b/jest-common/src/main/java/io/searchbox/cluster/State.java
@@ -9,7 +9,7 @@
  */
 public class State extends GenericResultAbstractAction {
 
-    public State(Builder builder) {
+    protected State(Builder builder) {
         super(builder);
         setURI(buildURI());
     }

diff --git a/jest-common/src/main/java/io/searchbox/cluster/UpdateSettings.java b/jest-common/src/main/java/io/searchbox/cluster/UpdateSettings.java
@@ -18,7 +18,7 @@
  */
 public class UpdateSettings extends GenericResultAbstractAction {
 
-    private UpdateSettings(Builder builder) {
+    protected UpdateSettings(Builder builder) {
         super(builder);
         setURI(buildURI());
         this.payload = builder.source;

diff --git a/jest-common/src/main/java/io/searchbox/core/Bulk.java b/jest-common/src/main/java/io/searchbox/core/Bulk.java
@@ -32,7 +32,7 @@ public class Bulk extends AbstractAction<BulkResult> {
     final static Logger log = LoggerFactory.getLogger(Bulk.class);
     protected Collection<BulkableAction> bulkableActions;
 
-    public Bulk(Builder builder) {
+    protected Bulk(Builder builder) {
         super(builder);
         indexName = builder.defaultIndex;
         typeName = builder.defaultType;

diff --git a/jest-common/src/main/java/io/searchbox/core/Count.java b/jest-common/src/main/java/io/searchbox/core/Count.java
@@ -12,7 +12,7 @@
  */
 public class Count extends AbstractAction<CountResult> {
 
-    public Count(Builder builder) {
+    protected Count(Builder builder) {
         super(builder);
 
         this.payload = builder.query;

diff --git a/jest-common/src/main/java/io/searchbox/core/Delete.java b/jest-common/src/main/java/io/searchbox/core/Delete.java
@@ -11,7 +11,7 @@
  */
 public class Delete extends GenericResultAbstractDocumentTargetedAction implements BulkableAction<JestResult> {
 
-    private Delete(Builder builder) {
+    protected Delete(Builder builder) {
         super(builder);
         setURI(buildURI());
     }

diff --git a/jest-common/src/main/java/io/searchbox/core/DeleteByQuery.java b/jest-common/src/main/java/io/searchbox/core/DeleteByQuery.java
@@ -11,7 +11,7 @@
  */
 public class DeleteByQuery extends GenericResultAbstractAction {
 
-    public DeleteByQuery(Builder builder) {
+    protected DeleteByQuery(Builder builder) {
         super(builder);
 
         this.payload = builder.query;

diff --git a/jest-common/src/main/java/io/searchbox/core/Explain.java b/jest-common/src/main/java/io/searchbox/core/Explain.java
@@ -10,7 +10,7 @@
  */
 public class Explain extends GenericResultAbstractDocumentTargetedAction {
 
-    private Explain(Builder builder) {
+    protected Explain(Builder builder) {
         super(builder);
         setURI(buildURI());
         this.payload = builder.query;

diff --git a/jest-common/src/main/java/io/searchbox/core/Get.java b/jest-common/src/main/java/io/searchbox/core/Get.java
@@ -9,7 +9,7 @@
  */
 public class Get extends GenericResultAbstractDocumentTargetedAction {
 
-    private Get(Builder builder) {
+    protected Get(Builder builder) {
         super(builder);
         setURI(buildURI());
     }

diff --git a/jest-common/src/main/java/io/searchbox/core/Index.java b/jest-common/src/main/java/io/searchbox/core/Index.java
@@ -15,7 +15,7 @@
  */
 public class Index extends GenericResultAbstractDocumentTargetedAction implements BulkableAction<JestResult> {
 
-    private Index(Builder builder) {
+    protected Index(Builder builder) {
         super(builder);
 
         this.payload = builder.source;

diff --git a/jest-common/src/main/java/io/searchbox/core/MoreLikeThis.java b/jest-common/src/main/java/io/searchbox/core/MoreLikeThis.java
@@ -10,7 +10,7 @@
  */
 public class MoreLikeThis extends GenericResultAbstractDocumentTargetedAction {
 
-    private MoreLikeThis(Builder builder) {
+    protected MoreLikeThis(Builder builder) {
         super(builder);
 
         this.payload = builder.query;

diff --git a/jest-common/src/main/java/io/searchbox/core/MultiGet.java b/jest-common/src/main/java/io/searchbox/core/MultiGet.java
@@ -25,12 +25,12 @@ protected MultiGet(AbstractAction.Builder builder) {
         setPathToResult("docs/_source");
     }
 
-    public MultiGet(Builder.ByDoc builder) {
+    protected MultiGet(Builder.ByDoc builder) {
         this((AbstractAction.Builder) builder);
         this.payload = ImmutableMap.of("docs", docsToMaps(builder.docs));
     }
 
-    public MultiGet(Builder.ById builder) {
+    protected MultiGet(Builder.ById builder) {
         this((AbstractAction.Builder) builder);
         this.payload = ImmutableMap.of("ids", builder.ids);
     }

diff --git a/jest-common/src/main/java/io/searchbox/core/MultiSearch.java b/jest-common/src/main/java/io/searchbox/core/MultiSearch.java
@@ -18,7 +18,7 @@ public class MultiSearch extends GenericResultAbstractAction {
 
     private Collection<Search> searches;
 
-    public MultiSearch(Builder builder) {
+    protected MultiSearch(Builder builder) {
         super(builder);
 
         this.searches = builder.searchList;

diff --git a/jest-common/src/main/java/io/searchbox/core/Percolate.java b/jest-common/src/main/java/io/searchbox/core/Percolate.java
@@ -12,7 +12,7 @@
  */
 public class Percolate extends GenericResultAbstractAction {
 
-    public Percolate(Builder builder) {
+    protected Percolate(Builder builder) {
         super(builder);
 
         this.indexName = builder.index;

diff --git a/jest-common/src/main/java/io/searchbox/core/Search.java b/jest-common/src/main/java/io/searchbox/core/Search.java
@@ -20,7 +20,7 @@ public class Search extends AbstractAction<SearchResult> {
     private String query;
     private List<Sort> sortList = new LinkedList<Sort>();
 
-    private Search(Builder builder) {
+    protected Search(Builder builder) {
         super(builder);
 
         this.query = builder.query;

diff --git a/jest-common/src/main/java/io/searchbox/core/SearchScroll.java b/jest-common/src/main/java/io/searchbox/core/SearchScroll.java
@@ -16,7 +16,7 @@ public class SearchScroll extends GenericResultAbstractAction {
     static final int MAX_SCROLL_ID_LENGTH = 1900;
     private final String restMethodName;
 
-    public SearchScroll(Builder builder) {
+    protected SearchScroll(Builder builder) {
         super(builder);
 
         if(builder.getScrollId().length() > MAX_SCROLL_ID_LENGTH) {

diff --git a/jest-common/src/main/java/io/searchbox/core/SearchShards.java b/jest-common/src/main/java/io/searchbox/core/SearchShards.java
@@ -11,7 +11,7 @@
  */
 public class SearchShards extends GenericResultAbstractAction {
 
-    private SearchShards(Builder builder) {
+    protected SearchShards(Builder builder) {
         super(builder);
         setURI(buildURI());
     }

diff --git a/jest-common/src/main/java/io/searchbox/core/Suggest.java b/jest-common/src/main/java/io/searchbox/core/Suggest.java
@@ -8,7 +8,7 @@
 
 public class Suggest extends AbstractAction<SuggestResult> {
 
-    private Suggest(final Builder builder) {
+    protected Suggest(final Builder builder) {
         super(builder);
 
         this.payload = builder.getQuery();

diff --git a/jest-common/src/main/java/io/searchbox/core/Update.java b/jest-common/src/main/java/io/searchbox/core/Update.java
@@ -12,7 +12,7 @@
  */
 public class Update extends GenericResultAbstractDocumentTargetedAction implements BulkableAction<JestResult> {
 
-    private Update(Builder builder) {
+    protected Update(Builder builder) {
         super(builder);
 
         this.payload = builder.payload;

diff --git a/jest-common/src/main/java/io/searchbox/core/Validate.java b/jest-common/src/main/java/io/searchbox/core/Validate.java
@@ -10,7 +10,7 @@
  */
 public class Validate extends GenericResultAbstractAction {
 
-    private Validate(Builder builder) {
+    protected Validate(Builder builder) {
         super(builder);
 
         this.indexName = builder.index;

diff --git a/jest-common/src/main/java/io/searchbox/indices/Analyze.java b/jest-common/src/main/java/io/searchbox/indices/Analyze.java
@@ -13,7 +13,7 @@
  */
 public class Analyze extends GenericResultAbstractAction {
 
-    public Analyze(Builder builder) {
+    protected Analyze(Builder builder) {
         super(builder);
 
         this.indexName = builder.index;

diff --git a/jest-common/src/main/java/io/searchbox/indices/ClearCache.java b/jest-common/src/main/java/io/searchbox/indices/ClearCache.java
@@ -9,7 +9,7 @@
  */
 public class ClearCache extends GenericResultAbstractAction {
 
-    public ClearCache(Builder builder) {
+    protected ClearCache(Builder builder) {
         super(builder);
         setURI(buildURI());
     }

diff --git a/jest-common/src/main/java/io/searchbox/indices/CloseIndex.java b/jest-common/src/main/java/io/searchbox/indices/CloseIndex.java
@@ -7,7 +7,7 @@
  */
 public class CloseIndex extends GenericResultAbstractAction {
 
-    public CloseIndex(Builder builder) {
+    protected CloseIndex(Builder builder) {
         super(builder);
 
         this.indexName = builder.index;

diff --git a/jest-common/src/main/java/io/searchbox/indices/CreateIndex.java b/jest-common/src/main/java/io/searchbox/indices/CreateIndex.java
@@ -12,7 +12,7 @@ public class CreateIndex extends GenericResultAbstractAction {
 
     private boolean isCreateOp = false;
 
-    public CreateIndex(Builder builder) {
+    protected CreateIndex(Builder builder) {
         super(builder);
 
         this.indexName = builder.index;

diff --git a/jest-common/src/main/java/io/searchbox/indices/DeleteIndex.java b/jest-common/src/main/java/io/searchbox/indices/DeleteIndex.java
@@ -8,7 +8,7 @@
  */
 public class DeleteIndex extends GenericResultAbstractAction {
 
-    public DeleteIndex(Builder builder) {
+    protected DeleteIndex(Builder builder) {
         super(builder);
         indexName = builder.index;
         typeName = builder.type;

diff --git a/jest-common/src/main/java/io/searchbox/indices/Flush.java b/jest-common/src/main/java/io/searchbox/indices/Flush.java
@@ -9,7 +9,7 @@
  */
 public class Flush extends GenericResultAbstractAction {
 
-    private Flush(Builder builder) {
+    protected Flush(Builder builder) {
         super(builder);
         setURI(buildURI());
     }

diff --git a/jest-common/src/main/java/io/searchbox/indices/IndicesExists.java b/jest-common/src/main/java/io/searchbox/indices/IndicesExists.java
@@ -10,7 +10,7 @@
  */
 public class IndicesExists extends GenericResultAbstractAction {
 
-    public IndicesExists(Builder builder) {
+    protected IndicesExists(Builder builder) {
         super(builder);
         setURI(buildURI());
     }

diff --git a/jest-common/src/main/java/io/searchbox/indices/OpenIndex.java b/jest-common/src/main/java/io/searchbox/indices/OpenIndex.java
@@ -7,7 +7,7 @@
  */
 public class OpenIndex extends GenericResultAbstractAction {
 
-    public OpenIndex(Builder builder) {
+    protected OpenIndex(Builder builder) {
         super(builder);
         this.indexName = builder.index;
         setURI(buildURI());

diff --git a/jest-common/src/main/java/io/searchbox/indices/Optimize.java b/jest-common/src/main/java/io/searchbox/indices/Optimize.java
@@ -9,7 +9,7 @@
  */
 public class Optimize extends GenericResultAbstractAction {
 
-    private Optimize(Builder builder) {
+    protected Optimize(Builder builder) {
         super(builder);
         setURI(buildURI());
     }

diff --git a/jest-common/src/main/java/io/searchbox/indices/Refresh.java b/jest-common/src/main/java/io/searchbox/indices/Refresh.java
@@ -9,7 +9,7 @@
  */
 public class Refresh extends GenericResultAbstractAction {
 
-    private Refresh(Builder builder) {
+    protected Refresh(Builder builder) {
         super(builder);
         setURI(buildURI());
     }

diff --git a/jest-common/src/main/java/io/searchbox/indices/Stats.java b/jest-common/src/main/java/io/searchbox/indices/Stats.java
@@ -11,7 +11,7 @@
  */
 public class Stats extends GenericResultAbstractAction {
 
-    private Stats(Builder builder) {
+    protected Stats(Builder builder) {
         super(builder);
         indexName = builder.getJoinedIndices();
 

diff --git a/jest-common/src/main/java/io/searchbox/indices/Status.java b/jest-common/src/main/java/io/searchbox/indices/Status.java
@@ -9,7 +9,7 @@
  */
 public class Status extends GenericResultAbstractAction {
 
-    private Status(Builder builder) {
+    protected Status(Builder builder) {
         super(builder);
         this.indexName = builder.getJoinedIndices();
         setURI(buildURI());

diff --git a/jest-common/src/main/java/io/searchbox/indices/aliases/AddAliasMapping.java b/jest-common/src/main/java/io/searchbox/indices/aliases/AddAliasMapping.java
@@ -7,10 +7,10 @@
  */
 public class AddAliasMapping extends AliasMapping {
 
-    private AddAliasMapping() {
+    protected AddAliasMapping() {
     }
 
-    private AddAliasMapping(Builder builder) {
+    protected AddAliasMapping(Builder builder) {
         this.indices.addAll(builder.indices);
         this.alias = builder.alias;
         this.filter = builder.filter;

diff --git a/jest-common/src/main/java/io/searchbox/indices/aliases/GetAliases.java b/jest-common/src/main/java/io/searchbox/indices/aliases/GetAliases.java
@@ -8,7 +8,7 @@
  */
 public class GetAliases extends GenericResultAbstractAction {
 
-    private GetAliases(Builder builder) {
+    protected GetAliases(Builder builder) {
         super(builder);
         setURI(buildURI());
     }

diff --git a/jest-common/src/main/java/io/searchbox/indices/aliases/ModifyAliases.java b/jest-common/src/main/java/io/searchbox/indices/aliases/ModifyAliases.java
@@ -15,7 +15,7 @@
  */
 public class ModifyAliases extends GenericResultAbstractAction {
 
-    private ModifyAliases(Builder builder) {
+    protected ModifyAliases(Builder builder) {
         super(builder);
 
         List<Map> actions = new LinkedList<Map>();

diff --git a/jest-common/src/main/java/io/searchbox/indices/aliases/RemoveAliasMapping.java b/jest-common/src/main/java/io/searchbox/indices/aliases/RemoveAliasMapping.java
@@ -7,10 +7,10 @@
  */
 public class RemoveAliasMapping extends AliasMapping {
 
-    private RemoveAliasMapping() {
+    protected RemoveAliasMapping() {
     }
 
-    private RemoveAliasMapping(Builder builder) {
+    protected RemoveAliasMapping(Builder builder) {
         this.indices.addAll(builder.indices);
         this.alias = builder.alias;
         this.filter = builder.filter;

diff --git a/jest-common/src/main/java/io/searchbox/indices/mapping/DeleteMapping.java b/jest-common/src/main/java/io/searchbox/indices/mapping/DeleteMapping.java
@@ -9,7 +9,7 @@
  */
 public class DeleteMapping extends GenericResultAbstractAction {
 
-    public DeleteMapping(Builder builder) {
+    protected DeleteMapping(Builder builder) {
         super(builder);
 
         this.indexName = builder.index;