Skip to content

[INLONG-10132][Audit] Clean up ElasticSearch and ClickHouse related code of audit-store #10134

Merged
dockerzhang merged 8 commits into
apache:masterfrom
doleyzi:INLONG-10132
May 8, 2024
Merged

[INLONG-10132][Audit] Clean up ElasticSearch and ClickHouse related code of audit-store #10134
dockerzhang merged 8 commits into
apache:masterfrom
doleyzi:INLONG-10132

Conversation

@doleyzi
Copy link
Copy Markdown
Contributor

@doleyzi doleyzi commented May 7, 2024

Motivation

At present, the Audit Store already has universal JDBC writing capabilities, so the customized size of ElasticSearch and ClickHouse can be removed.

Modifications

Clean up ElasticSearch and ClickHouse related code of audit-store.

@doleyzi doleyzi changed the title [INLONG-10132][Audit] Clean up Elasticsearch and ClickHouse related code of audit-store [INLONG-10132][Audit] Clean up ElasticSearch and ClickHouse related code of audit-store May 7, 2024
justinwwhuang
justinwwhuang previously approved these changes May 7, 2024
Copy link
Copy Markdown
Contributor

@justinwwhuang justinwwhuang left a comment

Choose a reason for hiding this comment

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

LGTM

luchunliang
luchunliang previously approved these changes May 7, 2024
Comment thread inlong-audit/audit-docker/Dockerfile Outdated
@doleyzi doleyzi dismissed stale reviews from luchunliang and justinwwhuang via b0f61a2 May 7, 2024 10:49
justinwwhuang
justinwwhuang previously approved these changes May 7, 2024
Comment thread inlong-audit/audit-docker/Dockerfile Outdated
Comment thread inlong-audit/audit-docker/Dockerfile Outdated
@dockerzhang dockerzhang merged commit aa674dc into apache:master May 8, 2024
herywang pushed a commit to herywang/inlong that referenced this pull request May 9, 2024
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.

[Improve][Audit] Clean up Elasticsearch and ClickHouse related code of audit-store

5 participants