[CALCITE-7510] EnumerableTableModify UPDATE, DELETE, INSERT support and bug fixes #4922
Annotations
8 errors and 8 warnings
|
Test
Execution failed for task ':core:autostyleJavaCheck'.
> The following files have format violations:
src/test/java/org/apache/calcite/adapter/enumerable/EnumerableTableModifyTest.java
@@ -38,16 +38,17 @@
sink.add(new Object[] {2, 20});
// Source row layout: [original_i, original_j, new_j].
-····final·List<Object[]>·source·=·Arrays.asList(␊
-········new·Object[]·{1,·10,·100},␊
+····final·List<Object[]>·source·=␊
+········Arrays.asList(new·Object[]·{1,·10,·100},␊
new Object[] {1, 10, 200});
-····final·long·count·=·EnumerableTableModify.applyUpdateOneToOne(␊
-········Linq4j.asEnumerable(source),·sink,·2,·new·int[]·{1});␊
+····final·long·count·=␊
+········EnumerableTableModify.applyUpdateOneToOne(Linq4j.asEnumerable(source),·sink,·2,·new·int[]·{1});␊
assertThat(count, is(2L));
assertThat(toValueRows(sink),
-········is(Arrays.asList(␊
+········is(␊
+············Arrays.asList(␊
Arrays.asList(1, 100),
Arrays.asList(1, 200),
Arrays.asList(1, 10),
@@ -61,14 +62,15 @@
sink.add(new Object[] {1, 10});
sink.add(new Object[] {2, 20});
-····final·List<Object[]>·source·=·Arrays.asList(␊
-········new·Object[]·{1,·10},␊
+····final·List<Object[]>·source·=␊
+········Arrays.asList(new·Object[]·{1,·10},␊
new Object[] {1, 10});
EnumerableTableModify.applyDeleteArrayRows(Linq4j.asEnumerable(source), sink);
assertThat(toValueRows(sink),
-········is(Arrays.asList(␊
+········is(␊
+············Arrays.asList(␊
Arrays.asList(1, 10),
Arrays.asList(2, 20))));
}
@@ -81,4 +83,3 @@
return valueRows;
}
}
-
You might want to adjust -PmaxCheckMessageLines=50
-PmaxFilesToList=10
-PminLinesPerFile=4
to see more violations
Run './gradlew autostyleApply' to fix the violations.
|
|
Test:
:core:autostyleJavaCheck#L0
Execution failed for task ':core:autostyleJavaCheck'.
Caused by:
The following files have format violations:
src/test/java/org/apache/calcite/adapter/enumerable/EnumerableTableModifyTest.java
@@ -38,16 +38,17 @@
sink.add(new Object[] {2, 20});
// Source row layout: [original_i, original_j, new_j].
-····final·List<Object[]>·source·=·Arrays.asList(␊
-········new·Object[]·{1,·10,·100},␊
+····final·List<Object[]>·source·=␊
+········Arrays.asList(new·Object[]·{1,·10,·100},␊
new Object[] {1, 10, 200});
-····final·long·count·=·EnumerableTableModify.applyUpdateOneToOne(␊
-········Linq4j.asEnumerable(source),·sink,·2,·new·int[]·{1});␊
+····final·long·count·=␊
+········EnumerableTableModify.applyUpdateOneToOne(Linq4j.asEnumerable(source),·sink,·2,·new·int[]·{1});␊
assertThat(count, is(2L));
assertThat(toValueRows(sink),
-········is(Arrays.asList(␊
+········is(␊
+············Arrays.asList(␊
Arrays.asList(1, 100),
Arrays.asList(1, 200),
Arrays.asList(1, 10),
@@ -61,14 +62,15 @@
sink.add(new Object[] {1, 10});
sink.add(new Object[] {2, 20});
-····final·List<Object[]>·source·=·Arrays.asList(␊
-········new·Object[]·{1,·10},␊
+····final·List<Object[]>·source·=␊
+········Arrays.asList(new·Object[]·{1,·10},␊
new Object[] {1, 10});
EnumerableTableModify.applyDeleteArrayRows(Linq4j.asEnumerable(source), sink);
assertThat(toValueRows(sink),
-········is(Arrays.asList(␊
+········is(␊
+············Arrays.asList(␊
Arrays.asList(1, 10),
Arrays.asList(2, 20))));
}
@@ -81,4 +83,3 @@
return valueRows;
}
}
-
You might want to adjust -PmaxCheckMessageLines=50
-PmaxFilesToList=10
-PminLinesPerFile=4
to see more violations
Run './gradlew autostyleApply' to fix the violations.
|
|
|
|
Test:
core/src/test/java/org/apache/calcite/adapter/enumerable/EnumerableTableModifyTest.java#L71
Replace 2 lines 71..71 with
is(
Arrays.asList(
|
|
Test:
core/src/test/java/org/apache/calcite/adapter/enumerable/EnumerableTableModifyTest.java#L64
Replace 2 lines 64..65 with
final List<Object[]> source =
Arrays.asList(new Object[] {1, 10},
|
|
Test:
core/src/test/java/org/apache/calcite/adapter/enumerable/EnumerableTableModifyTest.java#L50
Replace 2 lines 50..50 with
is(
Arrays.asList(
|
|
Test:
core/src/test/java/org/apache/calcite/adapter/enumerable/EnumerableTableModifyTest.java#L45
Replace 2 lines 45..46 with
final long count =
EnumerableTableModify.applyUpdateOneToOne(Linq4j.asEnumerable(source), sink, 2, new int[] {1});
|
|
Test:
core/src/test/java/org/apache/calcite/adapter/enumerable/EnumerableTableModifyTest.java#L41
Replace 2 lines 41..42 with
final List<Object[]> source =
Arrays.asList(new Object[] {1, 10, 100},
|
|
Complete job
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: burrunan/gradle-cache-action@v1. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Post Test
Failed to save: <h2>Our services aren't available right now</h2><p>We're working to restore all services as soon as possible. Please check back soon.</p>0ChUWagAAAABbABWmlyEjTpobbEgUYHVWUEhMMzBFREdFMDExNABFZGdl
|
|
Post Test
Failed to save: <h2>Our services aren't available right now</h2><p>We're working to restore all services as soon as possible. Please check back soon.</p>0ChUWagAAAABCzl/m5F6vToYcq3qpR6SKUEhMMzBFREdFMDExNABFZGdl
|
|
Post Test
Failed to save: <h2>Our services aren't available right now</h2><p>We're working to restore all services as soon as possible. Please check back soon.</p>09xQWagAAAADVHNr88AY7S6ZWU6jRui82UEhMMzBFREdFMDExNABFZGdl
|
|
Test
Failed to restore: Cache service responded with 400
|
|
Test
Failed to restore: Cache service responded with 400
|
|
Test
Failed to restore: Cache service responded with 400
|
|
Test
Failed to restore: Cache service responded with 400
|
background
wait
wait-all
cancel
Loading