Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: drop node target's key constraint requirement #2068

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

fbiville
Copy link
Collaborator

This blocks Community Edition (CE) users, as the only way to emulate key constraints is to combine unique constraints (available in CE) with existence constraints (available only in Enterprise Edition).

This blocks Community Edition (CE) users, as the only way to
emulate key constraints is to combine unique constraints
(available in CE) with existence constraints (available only in
Enterprise Edition).
Copy link

codecov bot commented Dec 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 46.14%. Comparing base (cd28d01) to head (efbcdc9).

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2068      +/-   ##
============================================
- Coverage     46.16%   46.14%   -0.02%     
+ Complexity     3843     3835       -8     
============================================
  Files           856      855       -1     
  Lines         51010    50996      -14     
  Branches       5365     5363       -2     
============================================
- Hits          23548    23532      -16     
- Misses        25760    25762       +2     
  Partials       1702     1702              
Components Coverage Δ
spanner-templates 67.88% <ø> (-0.02%) ⬇️
spanner-import-export 65.56% <ø> (-0.04%) ⬇️
spanner-live-forward-migration 76.42% <ø> (ø)
spanner-live-reverse-replication 77.35% <ø> (ø)
spanner-bulk-migration 87.16% <ø> (ø)

see 3 files with indirect coverage changes

Copy link
Contributor

@ali-ince ali-ince left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Are the tests failures related to this change?

@fbiville
Copy link
Collaborator Author

The test failures are all about the Neo4j container timing out:

2024-12-16T15:07:26.7348733Z [ERROR]   CustomNeo4jWriteQueryIT.setup:53 » ContainerLaunch Container startup failed fo...
2024-12-16T15:07:26.7350810Z [ERROR]   DataConversionIT.setUp:80 » ContainerLaunch Container startup failed for image...
2024-12-16T15:07:26.7352089Z [ERROR]   DataConversionIT.setUp:80 » ContainerLaunch Container startup failed for image...
2024-12-16T15:07:26.7353414Z [ERROR]   EscapingIT.setUp:64 » ContainerLaunch Container startup failed for image neo4j...
2024-12-16T15:07:26.7355413Z [ERROR]   EscapingIT.setUp:64 » ContainerLaunch Container startup failed for image neo4j...
2024-12-16T15:07:26.7357119Z [ERROR]   EscapingIT.setUp:64 » ContainerLaunch Container startup failed for image neo4j...
2024-12-16T15:07:26.7359182Z [ERROR]   InlineDataToNeo4jIT.setup:54 » ContainerLaunch Container startup failed for im...
2024-12-16T15:07:26.7360798Z [ERROR]   MovieImportIT.setup:55 » ContainerLaunch Container startup failed for image ne...
2024-12-16T15:07:26.7362633Z [ERROR]   SecretManagerIT.setup:57 » ContainerLaunch Container startup failed for image ...
2024-12-16T15:07:26.7364378Z [ERROR]   SinglePassGraphImportIT.setup:54 » ContainerLaunch Container startup failed fo...

One occurrence of this is:

2024-12-16T15:06:57.8099061Z [ERROR] runsTemplateWithNeo4jConnectionSecret(com.google.cloud.teleport.v2.neo4j.templates.SecretManagerIT)  Time elapsed: 240.683 s  <<< ERROR!
2024-12-16T15:06:57.8101143Z org.testcontainers.containers.ContainerLaunchException: Container startup failed for image neo4j:5-enterprise
2024-12-16T15:06:57.8102516Z 	at org.testcontainers.containers.GenericContainer.doStart(GenericContainer.java:359)
2024-12-16T15:06:57.8104983Z 	at org.testcontainers.containers.GenericContainer.start(GenericContainer.java:330)
2024-12-16T15:06:57.8106475Z 	at org.apache.beam.it.testcontainers.TestContainerResourceManager.<init>(TestContainerResourceManager.java:76)
2024-12-16T15:06:57.8109069Z 	at org.apache.beam.it.testcontainers.TestContainerResourceManager.<init>(TestContainerResourceManager.java:52)
2024-12-16T15:06:57.8110382Z 	at org.apache.beam.it.neo4j.Neo4jResourceManager.<init>(Neo4jResourceManager.java:84)
2024-12-16T15:06:57.8112436Z 	at org.apache.beam.it.neo4j.Neo4jResourceManager.<init>(Neo4jResourceManager.java:71)
2024-12-16T15:06:57.8114114Z 	at org.apache.beam.it.neo4j.Neo4jResourceManager.<init>(Neo4jResourceManager.java:48)
2024-12-16T15:06:57.8115735Z 	at org.apache.beam.it.neo4j.Neo4jResourceManager$Builder.build(Neo4jResourceManager.java:268)
2024-12-16T15:06:57.8118007Z 	at org.apache.beam.it.neo4j.Neo4jResourceManager$Builder.build(Neo4jResourceManager.java:200)
2024-12-16T15:06:57.8119997Z 	at com.google.cloud.teleport.v2.neo4j.templates.SecretManagerIT.setup(SecretManagerIT.java:57)
2024-12-16T15:06:57.8121296Z 	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
2024-12-16T15:06:57.8123149Z 	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
2024-12-16T15:06:57.8125819Z 	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
2024-12-16T15:06:57.8126261Z 	at java.base/java.lang.reflect.Method.invoke(Method.java:566)
2024-12-16T15:06:57.8128454Z 	at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:59)
2024-12-16T15:06:57.8129892Z 	at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
2024-12-16T15:06:57.8131639Z 	at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:56)
2024-12-16T15:06:57.8133323Z 	at org.junit.internal.runners.statements.RunBefores.invokeMethod(RunBefores.java:33)
2024-12-16T15:06:57.8134634Z 	at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24)
2024-12-16T15:06:57.8136219Z 	at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
2024-12-16T15:06:57.8138239Z 	at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:61)
2024-12-16T15:06:57.8138758Z 	at org.junit.runners.ParentRunner$3.evaluate(ParentRunner.java:306)
2024-12-16T15:06:57.8140635Z 	at org.junit.runners.BlockJUnit4ClassRunner$1.evaluate(BlockJUnit4ClassRunner.java:100)
2024-12-16T15:06:57.8141691Z 	at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:366)
2024-12-16T15:06:57.8143689Z 	at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:103)
2024-12-16T15:06:57.8145286Z 	at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:63)
2024-12-16T15:06:57.8146336Z 	at org.junit.runners.ParentRunner$4.run(ParentRunner.java:331)
2024-12-16T15:06:57.8148328Z 	at org.apache.maven.surefire.junitcore.pc.Scheduler$1.run(Scheduler.java:410)
2024-12-16T15:06:57.8149996Z 	at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
2024-12-16T15:06:57.8151241Z 	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
2024-12-16T15:06:57.8153747Z 	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
2024-12-16T15:06:57.8155344Z 	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
2024-12-16T15:06:57.8155821Z 	at java.base/java.lang.Thread.run(Thread.java:829)
2024-12-16T15:06:57.8158035Z Caused by: org.rnorth.ducttape.RetryCountExceededException: Retry limit hit with exception
2024-12-16T15:06:57.8159707Z 	at org.rnorth.ducttape.unreliables.Unreliables.retryUntilSuccess(Unreliables.java:88)
2024-12-16T15:06:57.8161351Z 	at org.testcontainers.containers.GenericContainer.doStart(GenericContainer.java:344)
2024-12-16T15:06:57.8161516Z 	... 32 more
2024-12-16T15:06:57.8163661Z Caused by: org.testcontainers.containers.ContainerLaunchException: Could not create/start container
2024-12-16T15:06:57.8165396Z 	at org.testcontainers.containers.GenericContainer.tryStart(GenericContainer.java:563)
2024-12-16T15:06:57.8167120Z 	at org.testcontainers.containers.GenericContainer.lambda$doStart$0(GenericContainer.java:354)
2024-12-16T15:06:57.8168843Z 	at org.rnorth.ducttape.unreliables.Unreliables.retryUntilSuccess(Unreliables.java:81)
2024-12-16T15:06:57.8168983Z 	... 33 more
2024-12-16T15:06:57.8170733Z Caused by: org.rnorth.ducttape.TimeoutException: java.util.concurrent.TimeoutException
2024-12-16T15:06:57.8171900Z 	at org.rnorth.ducttape.timeouts.Timeouts.callFuture(Timeouts.java:70)
2024-12-16T15:06:57.8173711Z 	at org.rnorth.ducttape.timeouts.Timeouts.doWithTimeout(Timeouts.java:60)
2024-12-16T15:06:57.8175783Z 	at org.testcontainers.containers.wait.strategy.WaitAllStrategy.waitUntilReady(WaitAllStrategy.java:54)
2024-12-16T15:06:57.8177996Z 	at org.testcontainers.containers.GenericContainer.waitUntilContainerStarted(GenericContainer.java:909)
2024-12-16T15:06:57.8179246Z 	at org.testcontainers.containers.GenericContainer.tryStart(GenericContainer.java:500)
2024-12-16T15:06:57.8179410Z 	... 35 more
2024-12-16T15:06:57.8180170Z Caused by: java.util.concurrent.TimeoutException
2024-12-16T15:06:57.8181820Z 	at java.base/java.util.concurrent.FutureTask.get(FutureTask.java:204)
2024-12-16T15:06:57.8183876Z 	at org.rnorth.ducttape.timeouts.Timeouts.callFuture(Timeouts.java:65)
2024-12-16T15:06:57.8184064Z 	... 39 more

Maybe a temporary issue on the CI side?
I don't see how the change in the PR could cause this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants