Skip to content

feat(logstore): support EnableLogStoreModify API - #223

Merged
crimson-gao merged 2 commits into
aliyun:masterfrom
Goclis:feat/enable-logstore-modify-api
Jul 28, 2026
Merged

feat(logstore): support EnableLogStoreModify API#223
crimson-gao merged 2 commits into
aliyun:masterfrom
Goclis:feat/enable-logstore-modify-api

Conversation

@Goclis

@Goclis Goclis commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add EnableLogStoreModifyRequest with the dedicated {"enabled": true} request body
  • expose request and convenience overloads through LogService and Client
  • send PUT /logstores/{logstore}/modification and return response headers through VoidResponse
  • add an offline unit test, usage sample, and functional regression test
  • document that request acceptance is asynchronous and repeated enable calls are idempotent

Compatibility

  • additive public API only; existing APIs and behavior are unchanged
  • remains compatible with Java 8
  • no dependency, artifact version, or release configuration changes

Testing

  • mvn -B -Dtest=EnableLogStoreModifyTest test
  • MAVEN_JAVA_HOME=/usr/lib/jvm/java-17-openjdk mvn -B package -DskipTests
  • MAVEN_JAVA_HOME=/usr/lib/jvm/java-1.8.0-openjdk mvn -B package -DskipTests
  • EnableLogStoreModifyFunctionTest passed against an isolated staging SLS environment; it creates and deletes temporary project/logstore resources

No credentials, account identifiers, or real endpoints are included in this change.

@CLAassistant

CLAassistant commented Jul 28, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@crimson-gao

Copy link
Copy Markdown
Collaborator

bump version +0.0.1

@Goclis

Goclis commented Jul 28, 2026

Copy link
Copy Markdown
Contributor Author

Updated the project version from 0.6.159 to 0.6.160 in commit 33cd85b.

Revalidated with:

  • mvn -B -Dtest=EnableLogStoreModifyTest test
  • JDK 17: mvn -B package -DskipTests
  • JDK 8: mvn -B package -DskipTests

Thanks for the review.

@crimson-gao
crimson-gao merged commit f1891f7 into aliyun:master Jul 28, 2026
2 checks passed
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.

3 participants