Tom Deering
Tom Deering
@prashanthjos sorry to take so long to respond. This reproduces it for me: ```java @Test public void testPropertyWalk() throws JsonProcessingException { final var someSchema = "{\n" + " \"$schema\": \"http://json-schema.org/draft-07/schema#\",\n"...
Likewise, adding a listener with `config.addPropertyWalkListener` collects even more unexpected things: ``` org.opentest4j.AssertionFailedError: Expected :[$.properties.tags] Actual :[$.properties.tags.type, $.properties.tags.items, $.properties.tags.items[0].type, $.type, $.$schema, $.description, $.title, $.properties] ```
:wave: Chiming in here as another consumer who was broken by this change. We had an optional property with schema `{type: array, minItems: 1}`, so version 7.5.0 of the generator...
@rdehuyss Thanks for the quick response! Unfortunately I don't have great repro steps for exactly what happened. Taking a look at one particular environment, it appears that the TABLE LOCKER...
Ok, if I'm reading the flow of that code right (and keeping in mind that multiple servers will be executing this logic concurrently): 1. Jobrunr determines the appropriate migration scripts...
Here's a thought. What if the SQLException [here](https://github.com/jobrunr/jobrunr/blob/v7.2.2/core/src/main/java/org/jobrunr/storage/sql/common/DatabaseCreator.java#L231), which is currently assumed to be because the table doesn't exist, were instead some other error (eg something transient due to networking)?...
Other than some telemetry showing that the service deployed at the time (makes sense), unfortunately there is nothing interesting in the logs. I do see that the infrastructure brought up...
Maybe it would be possible to somehow hash/record the migrations jobrunr thinks it needs to run on that locker row? If that row encodes nothing, or only migrations that were...
Hi @rdehuyss, I was out Thursday and Friday but am back today! > we've decided that for now, we will not allow JobRunr to start if this row is still...
> That makes sense to me! This at least would have allowed us to discover the issue at the time it happened rather than months later. I'm thinking through this...