Raúl Gracia

Results 87 comments of Raúl Gracia

Found another occurrence of: ``` io.pravega.controller.store.stream.ZKCounterTest > testCounter FAILED java.util.concurrent.CompletionException: java.lang.StackOverflowError at java.base/java.util.concurrent.CompletableFuture.encodeThrowable(CompletableFuture.java:314) at java.base/java.util.concurrent.CompletableFuture.uniComposeStage(CompletableFuture.java:1113) at java.base/java.util.concurrent.CompletableFuture.thenCompose(CompletableFuture.java:2235) ``` which is unrelated. Triggering again.

You are right @munish1789 the indentation confused me. If you could add spaces to the broken line, it would help.

@sachin-j-joshi I have done a manual test with this branch, and I still can reproduce the problem with BTreeIndex reading a corrupted footer if I manually add few bytes to...

@sachin-j-joshi if that optimization induces violation of atomicity, perhaps it shouldn't be allowed at all on any segment and should be just removed to simplify the codebase, right? Maybe we...

@sachin-j-joshi I have performed again the test scenario mentioned above, and now it works. I think that the problem was with the test approach when editing the attribute index file...

@sachin-j-joshi, please could you summarize what is the procedure in case of a rollback of a version of Pravega with this version and an older version of Pravega without it?...

According to our discussions, the exception we find with this test in Git Actions is a `StackOverflowException` caused when the test calls to `ZkInt96Counter.getNextCounter()`: https://github.com/pravega/pravega/blob/8e9986391177b5e3cb644facb2873f9a5f9d5491/controller/src/main/java/io/pravega/controller/store/stream/ZkInt96Counter.java#L69 The only reasonable explanation for...

@shshashwat can we close this one?

Thanks @eolivelli, reposting the answer in Slack for better visibility: https://pravega-io.slack.com/archives/C0151LSM46L/p1662115749173509 > is the ledger znode present on ZK ? The cluster is not in state anymore, but if the...