Skip to content

Commit

Permalink
review changes:
Browse files Browse the repository at this point in the history
- Got rid of END_REPO_DATA marker, replacing with simply writing the number of repos to read
- Avoid AtomicBoolean in `MetaFateStore.FateTxStoreImpl.push(repo)` by changing StackOverflowException to instead be a RuntimeException (previously Exception)
- Deleted unnecessary preexisting catch and immediate re-throw of a StackOverflowException in `MetaFateStore.FateTxStoreImpl.push(repo)`
- Cleaned up and refactored `MetaFateStore` methods which mutate existing FateData; now reuse same pattern across these methods: all call new method `MetaFateStore.mutate()`
  • Loading branch information
kevinrr888 committed Dec 9, 2024
1 parent 58869b0 commit 29edaa2
Show file tree
Hide file tree
Showing 2 changed files with 159 additions and 164 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
*/
package org.apache.accumulo.core.fate;

public class StackOverflowException extends Exception {
public class StackOverflowException extends RuntimeException {

public StackOverflowException(String msg) {
super(msg);
Expand Down
Loading

0 comments on commit 29edaa2

Please sign in to comment.