Skip to content

[AQA] ADS-3029#25

Open
mantasheva-dzh wants to merge 20 commits into
develop/4.3.0/1.10.1.1from
tests/4.3.0/1.10.1.1-3029
Open

[AQA] ADS-3029#25
mantasheva-dzh wants to merge 20 commits into
develop/4.3.0/1.10.1.1from
tests/4.3.0/1.10.1.1-3029

Conversation

@mantasheva-dzh

Copy link
Copy Markdown
Collaborator

No description provided.

@mantasheva-dzh mantasheva-dzh changed the title tests v3 [AQA] ADS-3029 Jun 19, 2026

<configuration>

<!-- 1. Настройка главной файловой системы (указываем ваш MinIO бакет) -->

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Translate to english or remove comments.

import software.amazon.awssdk.regions.Region;
import software.amazon.awssdk.services.s3.S3Client;

public class IcebergTableUtils {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Please rename this class and other classes in utils package. Avoid word "Utils" in class and package names, it's too generic.
Use more specific names, like IcebergTableClient, DockerClientService, etc.
Package can be renamed to 'service'

import org.apache.kafka.connect.data.Struct;
import org.apache.kafka.connect.json.JsonConverter;

public class Event extends BaseTestEvent {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Maybe use more specific name?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants