add feedback from Luca

packages/@aws-cdk/aws-glue-alpha/lib/database.ts

@@ -45,9 +45,9 @@ export interface DatabaseProps {
   readonly locationUri?: string;
 
   /**
-   * The description of the database.
+   * A description of the database.
    *
-   * @default - none.
+   * @default - no database description
    */
   readonly description?: string;
 }
@@ -94,12 +94,6 @@ export class Database extends Resource implements IDatabase {
    * Location URI of this database.
    */
   public readonly locationUri?: string;
-
-  /**
-   * Description of this database.
-   */
-  public readonly description?: string;
-
   constructor(scope: Construct, id: string, props: DatabaseProps = {}) {
     super(scope, id, {
       physicalName: props.databaseName ??
@@ -108,11 +102,13 @@ export class Database extends Resource implements IDatabase {
         }),
     });
 
-    this.description = props.description;
+    if (props.description !== undefined) {
+      validateDescription(props.description);
+    }
 
     let databaseInput: CfnDatabase.DatabaseInputProperty = {
       name: this.physicalName,
-      description: this.description,
+      description: props.description,
     };
 
     if (props.locationUri !== undefined) {
@@ -151,3 +147,9 @@ function validateLocationUri(locationUri: string): void {
     throw new Error(`locationUri length must be (inclusively) between 1 and 1024, but was ${locationUri.length}`);
   }
 }
+
+function validateDescription(description: string): void {
+  if (description.length > 2048) {
+    throw new Error(`description length must be less than or equal to 2048, but was ${description.length}`);
+  }
+}

packages/@aws-cdk/aws-glue-alpha/test/database.test.ts

@@ -111,6 +111,14 @@ test('locationUri length must be <= 1024', () => {
   ).toThrow();
 });
 
+test('description length must be <= 2048', () => {
+  expect(() =>
+    new glue.Database(stack, 'Database', {
+      description: 'a'.repeat(2049),
+    }),
+  ).toThrow();
+});
+
 test('can specify a physical name', () => {
   new glue.Database(stack, 'Database', {
     databaseName: 'my_database',