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

MariaDB Connector/J 3.0 - Binary Data Formatter Hex broken #16847

Closed
aa-matthias opened this issue Jun 21, 2022 · 5 comments · Fixed by #17662
Closed

MariaDB Connector/J 3.0 - Binary Data Formatter Hex broken #16847

aa-matthias opened this issue Jun 21, 2022 · 5 comments · Fixed by #17662

Comments

@aa-matthias
Copy link

System information:

  • Operating system (distribution) and version (Windows 11 21H2)
  • DBeaver version 22.1.0
  • Additional extensions

Connection specification:

  • Database name and version MariaDB 10.9.1
  • Driver name MariaDB Connector/J 3.0 (3.0.3, 3.0.4, 3.0.5)
  • Do you use tunnels or proxies (SSH, SOCKS, etc)? SSH

Describe the problem you're observing:

Setting Binary Format to Hex has no effect and data is not displayed correctly when using MariaDB Connector/J 3.0 - it's working fine with MariaDB Connector/J 2.0 (2.7.5)

Steps to reproduce, if exist:

Set Binary Display Format to Hex
Set Up a new Connection and specify any 3.. release from MariaDB Connector/J in the library config

Include any warning/errors/backtraces from the logs

@LonwoLonwo
Copy link
Member

Hello @aa-matthias

For now, I can't reproduce your issue.

Can you please provide a table DDL and sample data?
A video of the issue will also be beneficial.

@aa-matthias
Copy link
Author

Hi @LonwoLonwo

            CREATE TABLE IF NOT EXISTS `flow` (
                `id`                    BINARY(16)      NOT NULL,
                `name`                  VARCHAR(255)    COLLATE utf8mb4_unicode_ci  NOT NULL,
                `description`           MEDIUMTEXT      COLLATE utf8mb4_unicode_ci  NULL,
                `event_name`            VARCHAR(255)    NOT NULL,
                `priority`              INT(11)         NOT NULL DEFAULT 1,
                `payload`               LONGBLOB        NULL,
                `invalid`               TINYINT(1)      NOT NULL DEFAULT 0,
                `active`                TINYINT(1)      NOT NULL DEFAULT 0,
                `custom_fields`         JSON            NULL,
                `created_at`            DATETIME(3)     NOT NULL,
                `updated_at`            DATETIME(3)     NULL,
                PRIMARY KEY (`id`),
                INDEX `idx.flow.event_name` (`event_name`, `priority`),
                CONSTRAINT `json.flow.custom_fields` CHECK (JSON_VALID(`custom_fields`))
            ) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_unicode_ci;

INSERT INTO flow
(id, name, description, event_name, priority, payload, invalid, active, custom_fields, created_at, updated_at)
VALUES(0x03533CC8D7E54402BA0750AEFE5F560A, 'Payment enters status paid', NULL, 'state_enter.order_transaction.state.paid', 1, NULL, 0, 0, NULL, '2022-03-09 13:20:32.230', '2022-05-18 05:21:37.569');

image

@tgross35
Copy link

tgross35 commented Jun 28, 2022

How does dbeaver determine what driver JDBC driver to install by default? 2.7.1 is what the latest update installed for me, but that's from 2020. 3.0.5 is the latest https://mariadb.com/kb/en/mariadb-connector-j-releases/

@aa-matthias
Copy link
Author

@emironovDB - for me it's solved by updating to version 3.0.7 - the changelog mentions couple of changes related to binary display

https://github.com/mariadb-corporation/mariadb-connector-j/releases/tag/3.0.7

@emironovDB emironovDB modified the milestones: 22.2.0, 22.1.5 Aug 5, 2022
@LonwoLonwo
Copy link
Member

THanks for investigation everyone.

Probably we need to update default driver version for MariaDB.

@LonwoLonwo LonwoLonwo self-assigned this Aug 5, 2022
@LonwoLonwo LonwoLonwo modified the milestones: 22.1.5, 22.2.1 Aug 5, 2022
@emironovDB emironovDB added the sp:2 label Sep 5, 2022
@emironovDB emironovDB modified the milestones: 22.2.1, 22.2.2 Sep 6, 2022
@LonwoLonwo LonwoLonwo linked a pull request Sep 12, 2022 that will close this issue
@LonwoLonwo LonwoLonwo modified the milestones: 22.2.2, 22.2.1 Sep 12, 2022
@serge-rider serge-rider reopened this Sep 13, 2022
@uslss uslss self-assigned this Sep 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants