Skip to content

Enable to pass empty ArrayList in schemaless #20

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jx2lee
Copy link
Contributor

@jx2lee jx2lee commented Sep 13, 2024

like #19 , smt failed when empty ArrayList is contained.
my env is not using schema registry and same problem occurred, i fixed to pass transform when empty arraylist included

@jx2lee jx2lee changed the title Enable empty ArrayList when schemaless Enable to pass empty ArrayList when schemaless Sep 13, 2024
@jx2lee
Copy link
Contributor Author

jx2lee commented Sep 13, 2024

Error:  com.github.cedelsb.integration.Record2JsonStringIT  Time elapsed: 49.565 s  <<< ERROR!
org.testcontainers.containers.ContainerLaunchException: Local Docker Compose not found. Is docker-compose on the PATH?
	at com.github.cedelsb.integration.Record2JsonStringIT.setup(Record2JsonStringIT.java:106)

[INFO] 
[INFO] Results:
[INFO] 
Error:  Errors: 
Error:    Record2JsonStringIT.setup:106 » ContainerLaunch Local Docker Compose not found...
[INFO] 
Error:  Tests run: 1, Failures: 0, Errors: 1, Skipped: 0
[INFO] 
[INFO] 
[INFO] --- maven-failsafe-plugin:2.22.2:verify (default) @ kafka-connect-transform-record2jsonstring ---
[INFO] 
[INFO] ------------------------------------------------------------------------

test ci workflow failed 😞

@jx2lee jx2lee changed the title Enable to pass empty ArrayList when schemaless Enable to pass empty ArrayList in schemaless Sep 13, 2024
@jx2lee
Copy link
Contributor Author

jx2lee commented Nov 7, 2024

knock, knock!

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

Successfully merging this pull request may close these issues.

1 participant