>From Michael Blow <[email protected]>:
Michael Blow has uploaded this change for review. (
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20491?usp=email )
Change subject: [NO ISSUE][*DB][COMMON] Fix async checkpoint failure if only
metadata is flushed
......................................................................
[NO ISSUE][*DB][COMMON] Fix async checkpoint failure if only metadata is flushed
e.g.
java.util.NoSuchElementException
at o.a.a.c.utils.IntSortedBitSet$1.nextInt(IntSortedBitSet.java:219)
at o.a.a.c.utils.IntUtil.appendCompact(IntUtil.java:79)
at o.a.a.c.utils.Partitions.toString(Partitions.java:139)
at java.lang.String.valueOf(String.java:4465)
at
o.a.a.c.context.DatasetLifecycleManager.asyncFlushMatchingIndexes(DatasetLifecycleManager.java:526)
Ext-ref: MB-68946
Change-Id: I081c80a54a749e15a1b3e10c3a6391835679fff6
---
M
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/Partitions.java
1 file changed, 3 insertions(+), 1 deletion(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/91/20491/1
diff --git
a/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/Partitions.java
b/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/Partitions.java
index cc93b91..e939ce1 100644
---
a/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/Partitions.java
+++
b/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/Partitions.java
@@ -136,7 +136,9 @@
builder.append(',');
}
}
- IntUtil.appendCompact(iter, builder, MINUS_ONE);
+ if (iter.hasNext()) {
+ IntUtil.appendCompact(iter, builder, MINUS_ONE);
+ }
builder.append(']');
return builder.toString();
}
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20491?usp=email
To unsubscribe, or for help writing mail filters, visit
https://asterix-gerrit.ics.uci.edu/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: asterixdb
Gerrit-Branch: phoenix
Gerrit-Change-Id: I081c80a54a749e15a1b3e10c3a6391835679fff6
Gerrit-Change-Number: 20491
Gerrit-PatchSet: 1
Gerrit-Owner: Michael Blow <[email protected]>
