Skip to content

Commit 56c71a9

Browse files
mbouazizclaude
andcommitted
Keep .collect(Array) workaround until CI docker image is updated
The compiler fix for #769 is in specialize.sk, but the CI uses a docker image with the pre-built compiler. The workaround needs to stay in place until the docker image is rebuilt with the new compiler. Co-Authored-By: Claude Opus 4.5 <[email protected]>
1 parent 77317fa commit 56c71a9

1 file changed

Lines changed: 13 additions & 7 deletions

File tree

sql/src/SqlSchemaMigration.sk

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -16,14 +16,20 @@ private fun resetReactiveViews(
1616
options: Options,
1717
): void {
1818
viewsDir = context.unsafeGetEagerDir(getViewsDir(context).dirName);
19-
viewsToReset = viewsDir.unsafeGetFileIter().flatMap((key_fileiter) -> {
20-
(_key, fileiter) = key_fileiter;
21-
fileiter.filter((file) ->
22-
SelectFile::type(file).cselect.from.any((from) ->
23-
alteredDirs.contains(from.i0.name)
19+
// Note: .collect(Array) is a workaround for issue #769 (iterator compilation bug).
20+
// The compiler fix is in specialize.sk but the CI docker image needs to be updated
21+
// with the new compiler before this workaround can be removed.
22+
viewsToReset = viewsDir
23+
.unsafeGetFileIter()
24+
.flatMap((key_fileiter) -> {
25+
(_key, fileiter) = key_fileiter;
26+
fileiter.filter((file) ->
27+
SelectFile::type(file).cselect.from.any((from) ->
28+
alteredDirs.contains(from.i0.name)
29+
)
2430
)
25-
)
26-
});
31+
})
32+
.collect(Array);
2733

2834
for (childView in viewsToReset) {
2935
selectFile = SelectFile::type(childView);

0 commit comments

Comments
 (0)