Skip to content

Conversation

@brfrn169
Copy link
Collaborator

@brfrn169 brfrn169 commented Jun 4, 2025

Description

This PR fixes typo.

Related issues and/or PRs

N/A

Changes made

  • Fixed typo.

Checklist

The following is a best-effort checklist. If any items in this checklist are not applicable to this PR or are dependent on other, unmerged PRs, please still mark the checkboxes after you have read and understood each item.

  • I have commented my code, particularly in hard-to-understand areas.
  • I have updated the documentation to reflect the changes.
  • I have considered whether similar issues could occur in other products, components, or modules if this PR is for bug fixes.
  • Any remaining open issues linked to this PR are documented and up-to-date (Jira, GitHub, etc.).
  • Tests (unit, integration, etc.) have been added for the changes.
  • My changes generate no new warnings.
  • Any dependent changes in other PRs have been merged and published.

Additional notes (optional)

N/A

Release notes

N/A

@brfrn169 brfrn169 requested a review from Copilot June 4, 2025 05:11
@brfrn169 brfrn169 self-assigned this Jun 4, 2025
@brfrn169 brfrn169 added the bugfix label Jun 4, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes a typographical error in the exception and API documentation comments by replacing “nontranient” with “nontransient.”

  • Corrected the spelling of “nontransient” in exception class comments.
  • Updated API documentation comments in multiple interfaces.

Reviewed Changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
core/src/main/java/com/scalar/db/exception/transaction/ValidationException.java Corrected typo in the exception documentation comment.
core/src/main/java/com/scalar/db/exception/transaction/PreparationException.java Corrected typo in the exception documentation comment.
core/src/main/java/com/scalar/db/exception/transaction/CrudException.java Corrected typo in the exception documentation comment.
core/src/main/java/com/scalar/db/exception/transaction/CommitException.java Corrected typo in the exception documentation comment.
core/src/main/java/com/scalar/db/api/TwoPhaseCommitTransactionManager.java Corrected typo in the API documentation comment.
core/src/main/java/com/scalar/db/api/TwoPhaseCommitTransaction.java Corrected typos in multiple documentation comment lines.
core/src/main/java/com/scalar/db/api/TransactionManagerCrudOperable.java Corrected typos in multiple documentation comment lines.
core/src/main/java/com/scalar/db/api/TransactionCrudOperable.java Corrected typos in multiple documentation comment lines.
core/src/main/java/com/scalar/db/api/DistributedTransaction.java Corrected typo in the API documentation comment.

* An exception thrown when validating a transaction fails due to transient or nontransient faults.
* You can try retrying the transaction from the beginning, but the transaction may still fail if
* the cause is nontranient.
* the cause is nontransient.
Copy link

Copilot AI Jun 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo corrected: 'nontranient' has been updated to 'nontransient' for clarity in the documentation comment.

Copilot uses AI. Check for mistakes.
* An exception thrown when preparing a transaction fails due to transient or nontransient faults.
* You can try retrying the transaction from the beginning, but the transaction may still fail if
* the cause is nontranient.
* the cause is nontransient.
Copy link

Copilot AI Jun 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo corrected: 'nontranient' has been updated to 'nontransient' for clarity in the documentation comment.

Copilot uses AI. Check for mistakes.
* An exception thrown when a transaction CRUD operation fails due to transient or nontransient
* faults. You can try retrying the transaction from the beginning, but the transaction may still
* fail if the cause is nontranient.
* fail if the cause is nontransient.
Copy link

Copilot AI Jun 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo corrected: 'nontranient' has been updated to 'nontransient' in the exception documentation comment.

Copilot uses AI. Check for mistakes.
* An exception thrown when committing a transaction fails due to transient or nontransient faults.
* You can try retrying the transaction from the beginning, but the transaction may still fail if
* the cause is nontranient.
* the cause is nontransient.
Copy link

Copilot AI Jun 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo corrected: 'nontranient' has been updated to 'nontransient' in the exception documentation comment.

Copilot uses AI. Check for mistakes.
* @throws TransactionException if joining the transaction fails due to transient or nontransient
* faults. You can try retrying the transaction from the beginning, but the transaction may
* still fail if the cause is nontranient
* still fail if the cause is nontransient
Copy link

Copilot AI Jun 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo corrected: 'nontranient' has been updated to 'nontransient' for clarity in the API comment.

Copilot uses AI. Check for mistakes.
* @throws PreparationException if the transaction fails to prepare due to transient or
* nontransient faults. You can try retrying the transaction from the beginning, but the
* transaction may still fail if the cause is nontranient
* transaction may still fail if the cause is nontransient
Copy link

Copilot AI Jun 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo corrected: 'nontranient' has been updated to 'nontransient' in the documentation comment (additional similar corrections exist in this file).

Copilot uses AI. Check for mistakes.
* @throws CrudException if the transaction CRUD operation fails due to transient or nontransient
* faults. You can try retrying the transaction from the beginning, but the transaction may
* still fail if the cause is nontranient
* still fail if the cause is nontransient
Copy link

Copilot AI Jun 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Multiple occurrences of the typo 'nontranient' have been corrected to 'nontransient' in this file.

Copilot uses AI. Check for mistakes.
* @throws CrudException if the transaction CRUD operation fails due to transient or nontransient
* faults. You can try retrying the transaction from the beginning, but the transaction may
* still fail if the cause is nontranient
* still fail if the cause is nontransient
Copy link

Copilot AI Jun 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Multiple occurrences of the typo 'nontranient' have been corrected to 'nontransient' in this file.

Copilot uses AI. Check for mistakes.
* @throws CommitException if the transaction fails to commit due to transient or nontransient
* faults. You can try retrying the transaction from the beginning, but the transaction may
* still fail if the cause is nontranient
* still fail if the cause is nontransient
Copy link

Copilot AI Jun 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo corrected: 'nontranient' has been updated to 'nontransient' for clarity in the documentation comment.

Copilot uses AI. Check for mistakes.
Copy link
Contributor

@feeblefakie feeblefakie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thank you!

Copy link
Contributor

@komamitsu komamitsu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 👍

@brfrn169 brfrn169 merged commit c303b05 into master Jun 4, 2025
56 checks passed
@brfrn169 brfrn169 deleted the fix-typo branch June 4, 2025 11:12
feeblefakie pushed a commit that referenced this pull request Jun 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants