Merge pull request #197 from knihit/master

Fix for Issue #196. Changed S3 protocol to use s3a:// for storing transformed output data

BREAKING CHANGE: any testing snapshots will need to be refreshed.

Author: biffgaut

.viperlightignore

@@ -121,3 +121,5 @@ source/patterns/@aws-solutions-constructs/aws-lambda-sagemakerendpoint/test/test
 source/patterns/@aws-solutions-constructs/aws-lambda-sagemakerendpoint/test/test-helper.ts:78
 source/patterns/@aws-solutions-constructs/aws-lambda-sagemakerendpoint/test/test-helper.ts:81
 source/patterns/@aws-solutions-constructs/aws-lambda-sagemakerendpoint/test/test-helper.ts:84
+# This is a test case
+source/patterns/@aws-solutions-constructs/aws-kinesisstreams-gluejob/test/test.kinesisstream-gluejob.test.ts:129

source/patterns/@aws-solutions-constructs/aws-kinesisstreams-gluejob/lib/index.ts

@@ -156,7 +156,7 @@ export class KinesisstreamsToGluejob extends Construct {
       outputDataStore: props.outputDataStore!
     });
 
-    this.glueJobRole = this.buildRolePolicy(scope, this.database, this.table, this.glueJob, this.glueJobRole);
+    this.glueJobRole = this.buildRolePolicy(scope, id, this.database, this.table, this.glueJob, this.glueJobRole);
   }
 
   /**
@@ -168,8 +168,9 @@ export class KinesisstreamsToGluejob extends Construct {
    * @param glueJob
    * @param role
    */
-  private buildRolePolicy(scope: Construct, glueDatabase: glue.CfnDatabase, glueTable: glue.CfnTable, glueJob: glue.CfnJob, role: IRole): IRole {
-    const _glueJobPolicy = new Policy(scope, 'GlueJobPolicy', {
+  private buildRolePolicy(scope: Construct, id: string, glueDatabase: glue.CfnDatabase, glueTable: glue.CfnTable,
+    glueJob: glue.CfnJob, role: IRole): IRole {
+    const _glueJobPolicy = new Policy(scope, `${id}GlueJobPolicy`, {
       statements: [ new PolicyStatement({
         effect: Effect.ALLOW,
         actions: [ 'glue:GetJob' ],