Conversation
# Conflicts: # ice-rest-catalog/src/main/java/com/altinity/ice/rest/catalog/Main.java
|
The testcontainers version we have is too old, I'm getting a Docker client version error |
xieandrew
left a comment
xieandrew
left a comment
There was a problem hiding this comment.
Some initial comments from a first look
ice-rest-catalog/src/test/java/com/altinity/ice/rest/catalog/RESTCatalogIT.java
Outdated
Show resolved
Hide resolved
ice-rest-catalog/src/test/java/com/altinity/ice/rest/catalog/RESTCatalogInsertIT.java
Outdated
Show resolved
Hide resolved
| echo "✓ Created namespace: ${NAMESPACE_NAME}" | ||
|
|
||
| # List namespaces to verify | ||
| {{ICE_CLI}} --config {{CLI_CONFIG}} list-namespaces |
There was a problem hiding this comment.
This command doesn't work:
Unmatched argument at index 2: 'list-namespaces'
Did you mean: ice delete-namespace or ice describe?
| * | ||
| * <p>This class uses Jackson/SnakeYAML annotations for YAML deserialization. | ||
| */ | ||
| public class ScenarioConfig { |
There was a problem hiding this comment.
Is it possible to use records for Jackson ObjectMapper? Would reduce boilerplate
| } | ||
|
|
||
| /** Result of executing a single script. */ | ||
| public static class ScriptExecutionResult { |
|
ToDo: |
2 participants
Integration test using template