Skip to content

Commit 51d83f9

Browse files
committed
fix: fix migration analyzer for arangosearch views not checking consolidationIntervalMsec property
If consolidationIntervalMsec differed from the configured value, we previously did not report a pending migration. If the update-view-migration had been detected due to a different reason and it had been run, the property would have been reset though. Now, the property is correctly checked and updated.
1 parent a8e0f68 commit 51d83f9

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

src/database/arangodb/schema-migration/arango-search-helpers.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -246,6 +246,10 @@ export function isEqualProperties(
246246
isEqual(
247247
definitionProperties.commitIntervalMsec,
248248
(viewProperties as any).commitIntervalMsec /* somehow missing in types */,
249+
) &&
250+
isEqual(
251+
definitionProperties.consolidationIntervalMsec,
252+
viewProperties.consolidationIntervalMsec,
249253
)
250254
);
251255
}

0 commit comments

Comments
 (0)