Allow passing explicit null values for optional input members (#1542)

This makes it easier to define the array literal for input objects as
PHP does not have a syntax sugar for conditional keys in an array
literal. This is consistent with the generated code for value objects.

Even though input objects allow to omit required members in the
constructor shape (as they can be set later by using the setter), the
phpdoc type still does not allow passing null explicitly (even though
the code would deal with it until the validation run) so that static
analysis tools can catch mistakes there. Passing a required member
explicitly is intended to pass a valid value for it and not a
potentially missing one.

Author: stof

src/Input/BatchExecuteStatementRequest.php

@@ -91,10 +91,10 @@ final class BatchExecuteStatementRequest extends Input
      *   resourceArn?: string,
      *   secretArn?: string,
      *   sql?: string,
-     *   database?: string,
-     *   schema?: string,
-     *   parameterSets?: array[],
-     *   transactionId?: string,
+     *   database?: null|string,
+     *   schema?: null|string,
+     *   parameterSets?: null|array[],
+     *   transactionId?: null|string,
      *   '@region'?: string|null,
      * } $input
      */
@@ -117,10 +117,10 @@ public function __construct(array $input = [])
      *   resourceArn?: string,
      *   secretArn?: string,
      *   sql?: string,
-     *   database?: string,
-     *   schema?: string,
-     *   parameterSets?: array[],
-     *   transactionId?: string,
+     *   database?: null|string,
+     *   schema?: null|string,
+     *   parameterSets?: null|array[],
+     *   transactionId?: null|string,
      *   '@region'?: string|null,
      * }|BatchExecuteStatementRequest $input
      */

src/Input/BeginTransactionRequest.php

@@ -48,8 +48,8 @@ final class BeginTransactionRequest extends Input
      * @param array{
      *   resourceArn?: string,
      *   secretArn?: string,
-     *   database?: string,
-     *   schema?: string,
+     *   database?: null|string,
+     *   schema?: null|string,
      *   '@region'?: string|null,
      * } $input
      */
@@ -66,8 +66,8 @@ public function __construct(array $input = [])
      * @param array{
      *   resourceArn?: string,
      *   secretArn?: string,
-     *   database?: string,
-     *   schema?: string,
+     *   database?: null|string,
+     *   schema?: null|string,
      *   '@region'?: string|null,
      * }|BeginTransactionRequest $input
      */

src/Input/ExecuteStatementRequest.php

@@ -127,14 +127,14 @@ final class ExecuteStatementRequest extends Input
      *   resourceArn?: string,
      *   secretArn?: string,
      *   sql?: string,
-     *   database?: string,
-     *   schema?: string,
-     *   parameters?: array<SqlParameter|array>,
-     *   transactionId?: string,
-     *   includeResultMetadata?: bool,
-     *   continueAfterTimeout?: bool,
-     *   resultSetOptions?: ResultSetOptions|array,
-     *   formatRecordsAs?: RecordsFormatType::*,
+     *   database?: null|string,
+     *   schema?: null|string,
+     *   parameters?: null|array<SqlParameter|array>,
+     *   transactionId?: null|string,
+     *   includeResultMetadata?: null|bool,
+     *   continueAfterTimeout?: null|bool,
+     *   resultSetOptions?: null|ResultSetOptions|array,
+     *   formatRecordsAs?: null|RecordsFormatType::*,
      *   '@region'?: string|null,
      * } $input
      */
@@ -159,14 +159,14 @@ public function __construct(array $input = [])
      *   resourceArn?: string,
      *   secretArn?: string,
      *   sql?: string,
-     *   database?: string,
-     *   schema?: string,
-     *   parameters?: array<SqlParameter|array>,
-     *   transactionId?: string,
-     *   includeResultMetadata?: bool,
-     *   continueAfterTimeout?: bool,
-     *   resultSetOptions?: ResultSetOptions|array,
-     *   formatRecordsAs?: RecordsFormatType::*,
+     *   database?: null|string,
+     *   schema?: null|string,
+     *   parameters?: null|array<SqlParameter|array>,
+     *   transactionId?: null|string,
+     *   includeResultMetadata?: null|bool,
+     *   continueAfterTimeout?: null|bool,
+     *   resultSetOptions?: null|ResultSetOptions|array,
+     *   formatRecordsAs?: null|RecordsFormatType::*,
      *   '@region'?: string|null,
      * }|ExecuteStatementRequest $input
      */

src/RdsDataServiceClient.php

@@ -54,10 +54,10 @@ class RdsDataServiceClient extends AbstractApi
      *   resourceArn: string,
      *   secretArn: string,
      *   sql: string,
-     *   database?: string,
-     *   schema?: string,
-     *   parameterSets?: array[],
-     *   transactionId?: string,
+     *   database?: null|string,
+     *   schema?: null|string,
+     *   parameterSets?: null|array[],
+     *   transactionId?: null|string,
      *   '@region'?: string|null,
      * }|BatchExecuteStatementRequest $input
      *
@@ -101,8 +101,8 @@ public function batchExecuteStatement($input): BatchExecuteStatementResponse
      * @param array{
      *   resourceArn: string,
      *   secretArn: string,
-     *   database?: string,
-     *   schema?: string,
+     *   database?: null|string,
+     *   schema?: null|string,
      *   '@region'?: string|null,
      * }|BeginTransactionRequest $input
      *
@@ -180,14 +180,14 @@ public function commitTransaction($input): CommitTransactionResponse
      *   resourceArn: string,
      *   secretArn: string,
      *   sql: string,
-     *   database?: string,
-     *   schema?: string,
-     *   parameters?: array<SqlParameter|array>,
-     *   transactionId?: string,
-     *   includeResultMetadata?: bool,
-     *   continueAfterTimeout?: bool,
-     *   resultSetOptions?: ResultSetOptions|array,
-     *   formatRecordsAs?: RecordsFormatType::*,
+     *   database?: null|string,
+     *   schema?: null|string,
+     *   parameters?: null|array<SqlParameter|array>,
+     *   transactionId?: null|string,
+     *   includeResultMetadata?: null|bool,
+     *   continueAfterTimeout?: null|bool,
+     *   resultSetOptions?: null|ResultSetOptions|array,
+     *   formatRecordsAs?: null|RecordsFormatType::*,
      *   '@region'?: string|null,
      * }|ExecuteStatementRequest $input
      *