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

Is insert with null value generate wrong sql query ? #271

Closed
mmaryo opened this issue Jul 14, 2022 · 2 comments
Closed

Is insert with null value generate wrong sql query ? #271

mmaryo opened this issue Jul 14, 2022 · 2 comments
Labels
status: invalid An issue that we don't feel is valid

Comments

@mmaryo
Copy link

mmaryo commented Jul 14, 2022

Bug Report

It looks like a bug

CryptoExchanges(null, null) Should generates this request : INSERT INTO "common"."CryptoExchanges" VALUES (DEFAULT, DEFAULT);

But I see it generates: INSERT INTO "common"."CryptoExchanges" VALUES (DEFAULT)

With an error : Caused by: org.h2.jdbc.JdbcSQLSyntaxErrorException: Column count does not match; SQL statement:

image

I join a project with last version of Spring boot, Java, Kotlin and H2 R2dbc

You can find a test inside that generate this error
And an SQL script for init the DB

    @Test
    fun findByExchangeCodeNotNull() {

        // given
        val uni = CryptoExchanges(null, "uni")
        val null1 = CryptoExchanges(null, null)
        cryptoExchangesRepository.saveAll(listOf(uni, null1)).collectList().block()

        // when
        val res = cryptoExchangesRepository.findByExchangeCodeNotNull()

        // then
        res.`as`(StepVerifier::create)
            .expectNextMatches { it.id == 1L && it.exchangeCode == "uni" }
            .expectNextMatches { it.id == 2L && it.exchangeCode == null}
            .verifyComplete()
    }



    CREATE SCHEMA "common";
    
    CREATE TABLE "common"."CryptoExchanges"
    (
        "Id"           SERIAL NOT NULL,
        "ExchangeCode" character varying(30)
    );
    
    INSERT INTO "common"."CryptoExchanges" VALUES (DEFAULT, 'crypto');
    
    INSERT INTO "common"."CryptoExchanges" VALUES (DEFAULT, DEFAULT);

demo.zip

Best

@mp911de
Copy link
Member

mp911de commented Jul 14, 2022

This issue tracker hosts bug reports and feature requests for the R2DBC specification. Looking at your code, it seems related to Spring Data.

@mp911de mp911de closed this as not planned Won't fix, can't repro, duplicate, stale Jul 14, 2022
@mp911de mp911de added the status: invalid An issue that we don't feel is valid label Jul 14, 2022
@mmaryo
Copy link
Author

mmaryo commented Jul 14, 2022

@mp911de thx I write to spring data now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: invalid An issue that we don't feel is valid
Projects
None yet
Development

No branches or pull requests

2 participants