Skip to content
This repository has been archived by the owner on Jun 7, 2024. It is now read-only.

Commit

Permalink
Change swapped config items
Browse files Browse the repository at this point in the history
usergrid.cluster.region* - Akka cluster region config
usergrid.queue.region* - SQS region config
  • Loading branch information
mdunker committed Dec 6, 2017
1 parent 4a65910 commit cda4d20
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ public interface EntityManagerFig extends GuicyFig {
* Comma-separated list of one or more Amazon regions to use if multiregion
* is set to true.
*/
@Key( "usergrid.queue.regionList" )
@Key( "usergrid.cluster.region.list" )
@Default("us-east-1")
String getRegionList();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,14 +14,14 @@ public interface LegacyQueueFig extends GuicyFig {
* http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/using-regions-availability-zones.html*
*/

String USERGRID_CLUSTER_REGION_LIST = "usergrid.cluster.region.list";
String USERGRID_CLUSTER_REGION_LOCAL = "usergrid.cluster.region.local";
String USERGRID_QUEUE_REGION_LIST = "usergrid.queue.regionList";
String USERGRID_QUEUE_REGION_LOCAL = "usergrid.queue.region";


/**
* Primary region to use for Amazon queues.
*/
@Key( USERGRID_CLUSTER_REGION_LOCAL )
@Key(USERGRID_QUEUE_REGION_LOCAL)
@Default("us-east-1")
String getPrimaryRegion();

Expand All @@ -37,7 +37,7 @@ public interface LegacyQueueFig extends GuicyFig {
* Comma-separated list of one or more Amazon regions to use if multiregion
* is set to true.
*/
@Key( USERGRID_CLUSTER_REGION_LIST )
@Key(USERGRID_QUEUE_REGION_LIST)
@Default("us-east-1")
String getRegionList();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -248,7 +248,7 @@ private String setupTopics( final String queueName ) throws Exception {
region = Region.getRegion(regions);
}
catch (IllegalArgumentException e) {
throw new IllegalArgumentException("INVALID REGION FROM CONFIGURATION " + LegacyQueueFig.USERGRID_CLUSTER_REGION_LIST + ": " + regionName, e);
throw new IllegalArgumentException("INVALID REGION FROM CONFIGURATION " + LegacyQueueFig.USERGRID_QUEUE_REGION_LIST + ": " + regionName, e);
}

AmazonSQSClient sqsClient = createSQSClient( region );
Expand Down Expand Up @@ -825,7 +825,7 @@ private Region getRegion() {
return Region.getRegion(regions);
}
catch (IllegalArgumentException e) {
throw new IllegalArgumentException("INVALID PRIMARY REGION FROM CONFIGURATION " + LegacyQueueFig.USERGRID_CLUSTER_REGION_LOCAL + ": " + regionName, e);
throw new IllegalArgumentException("INVALID PRIMARY REGION FROM CONFIGURATION " + LegacyQueueFig.USERGRID_QUEUE_REGION_LOCAL + ": " + regionName, e);
}
}

Expand Down

1 comment on commit cda4d20

@peterj99a
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Please sign in to comment.