aws-cdk icon indicating copy to clipboard operation
aws-cdk copied to clipboard

(aws-kinesisfirehose): Record format conversion with AWS Glue for DeliveryStream S3 destination

Open madeline-k opened this issue 3 years ago • 4 comments

Allow record format conversion for Kinesis Data Firehose delivery streams as described here: https://docs.aws.amazon.com/firehose/latest/dev/record-format-conversion.html

Part of https://github.com/aws/aws-cdk/issues/7536

Use Case

Converting the format of input data for a delivery stream from JSON to Apache Parquet or Apache ORC.

Proposed Solution

See RFC 340: Kinesis Firehose Delivery Stream

And this branch with the prototype

  • [ ] :wave: I may be able to implement this feature request
  • [ ] :warning: This feature might incur a breaking change

This is a :rocket: Feature Request

madeline-k avatar Jul 11 '21 22:07 madeline-k

@madeline-k with this being a year old, is the PR still relevant? are there processes holding it up?

shellscape avatar Aug 24 '22 20:08 shellscape

@madeline-k Is work being done on this issue. This seems to be a long pending and inevitable feature for kinesis firehose.

adeelamin15 avatar Jul 11 '23 05:07 adeelamin15

There was another construct that had this functionality and it is not longer available: aws_kinesisstreams_kinesisfirehose_s3

You could define a set of processors:

          processingConfiguration: {
            enabled: true,
            processors: [
              {
                type: 'MetadataExtraction',
                parameters: [
                  {
                    parameterName: 'MetadataExtractionQuery',
                    parameterValue: '{tablename: .tableName}',
                  },
                  {
                    parameterName: 'JsonParsingEngine',
                    parameterValue: 'JQ-1.6',
                  },
                ],
              },
              {
                type: 'AppendDelimiterToRecord',
                parameters: [
                  {
                    parameterName: 'Delimiter',
                    parameterValue: '\\n',
                  },
                ],
              },
            ],
          }
        }

guyernest avatar Oct 15 '23 18:10 guyernest

What is the status of this feature in the L2? The L1 supports this.

Xenoha avatar May 08 '24 19:05 Xenoha

This issue has received a significant amount of attention so we are automatically upgrading its priority. A member of the community will see the re-prioritization and provide an update on the issue.

github-actions[bot] avatar Jun 16 '24 00:06 github-actions[bot]