gemini-code-assist[bot] commented on code in PR #38492: URL: https://github.com/apache/beam/pull/38492#discussion_r3237240251
########## sdks/java/io/google-cloud-platform/src/main/java/org/apache/beam/sdk/io/gcp/spanner/SpannerIO.java: ########## @@ -311,7 +311,7 @@ * * <p>Note that the <a * href="https://cloud.google.com/spanner/quotas#limits_for_creating_reading_updating_and_deleting_data";>maximum - * size of a single transaction</a> is 20,000 mutated cells - including cells in indexes. If you + * size of a single transaction</a> is 80,000 mutated cells - including cells in indexes. If you Review Comment:  The documentation update to 80,000 is correct, but the change is incomplete. The `DEFAULT_MAX_NUM_MUTATIONS` constant in this class should also be updated to 80,000 to ensure the default behavior matches the documentation. Additionally, consider using 'number of mutations' instead of 'size' to avoid confusion with the separate 100MB commit size limit. Finally, please update the PR description to remove the placeholder text and fix the typo ('transacton'). ```suggestion * number of mutations in a single transaction</a> is 80,000 mutations - including mutations in indexes. If you ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
