aws / aws-cdk

The AWS Cloud Development Kit is a framework for defining cloud infrastructure in code
https://aws.amazon.com/cdk
Apache License 2.0
11.68k stars 3.93k forks source link

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

Open madeline-k opened 3 years ago

madeline-k commented 3 years ago

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


This is a :rocket: Feature Request

shellscape commented 2 years ago

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

adeelamin15 commented 1 year ago

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

guyernest commented 1 year ago

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',
                  },
                ],
              },
            ],
          }
        }
Xenoha commented 6 months ago

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

github-actions[bot] commented 5 months ago

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.