GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — refactor: 1.0.0 API by mrrtree · Pull Request #40 · apache/horaedb-client-java

This repository was archived by the owner on Mar 12, 2026. It is now read-only.

Conversation

@mrrtree

@mrrtree mrrtree commented Feb 7, 2023

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Closes #

Rationale for this change

What changes are included in this PR?

  • SDK refactor query/write API
  • Remove management client side, replaced by generic SqlQuery
  • CeresDBClient builder add required parameter RouteMode

Are there any user-facing changes?

same as changes

How does this change test

All unit test passed.
Complete data life process test passed.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
xvyang.xy added 9 commits February 6, 2023 14:57
xvyang.xy
feat: remove http mangement
1c0be46
xvyang.xy
refactor: refactor sdk api
791e445
xvyang.xy
chore: remove dependency arvo
2b9ce99
xvyang.xy
doc: update docs with new api
7184c9c
xvyang.xy
chore: rename metric to table in RouteClient
5573d29
xvyang.xy
refactor: rename RouteMode
eff319b
xvyang.xy
feat: supprt zstd batch decompress
f3286a7
xvyang.xy
doc: update version to 1.0.0.alpha
c99f831
xvyang.xy
refactor: rename Value float64 to double
4d1136f
@CLAassistant

CLAassistant commented Feb 7, 2023 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

xvyang.xy added 3 commits February 8, 2023 15:23
xvyang.xy
fix: fix WriteOkTest assert
4e07f25
xvyang.xy
chore: license header instread by simple one
7a698a9
xvyang.xy
doc: update changelog 1.0.0.alpha
4c573f2
ZuLiangWang
ZuLiangWang reviewed Feb 8, 2023
View reviewed changes
Comment thread README.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread README.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread README.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread README.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread README_CN.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread ceresdb-example/src/test/java/io/ceresdb/ReadmeTest.java Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread ceresdb-protocol/pom.xml Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread ceresdb-protocol/src/main/java/io/ceresdb/models/Point.java Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread ceresdb-protocol/src/main/java/io/ceresdb/models/Row.java Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread docs/write.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

ZuLiangWang
ZuLiangWang reviewed Feb 8, 2023
View reviewed changes
Comment thread docs/CHANGELOG.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

@mrrtree mrrtree changed the title Feat/refactor api refactor: refactor 1.0.0 api Feb 9, 2023
MichaelLeeHZ
MichaelLeeHZ reviewed Feb 9, 2023
View reviewed changes
Comment thread README.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

MichaelLeeHZ
MichaelLeeHZ reviewed Feb 9, 2023
View reviewed changes
Comment thread README_CN.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

MichaelLeeHZ
MichaelLeeHZ reviewed Feb 9, 2023
View reviewed changes
Comment thread ceresdb-example/src/test/java/io/ceresdb/ReadmeTest.java Resolved

Uh oh!

There was an error while loading. Please reload this page.

MichaelLeeHZ
MichaelLeeHZ reviewed Feb 9, 2023
View reviewed changes
Comment thread ceresdb-protocol/src/main/java/io/ceresdb/options/CeresDBOptions.java Resolved

Uh oh!

There was an error while loading. Please reload this page.

MichaelLeeHZ
MichaelLeeHZ reviewed Feb 9, 2023
View reviewed changes
Comment thread README_CN.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

@mrrtree
mrrtree force-pushed the feat/refactor_api branch 2 times, most recently from 817e92f to 9b19ac4 Compare February 9, 2023 09:23
xvyang.xy
oc: tranlate doc/* to en
7ba21fb
@mrrtree
mrrtree force-pushed the feat/refactor_api branch from 9b19ac4 to 7ba21fb Compare February 9, 2023 09:29
xvyang.xy added 2 commits February 9, 2023 17:47
xvyang.xy
fix: use simple license header
44d20bb
xvyang.xy
feat: support PointBuilder
34a654c
@mrrtree mrrtree changed the title refactor: refactor 1.0.0 api refactor: 1.0.0 api Feb 10, 2023
@mrrtree mrrtree changed the title refactor: 1.0.0 api refactor: 1.0.0 API Feb 10, 2023
xvyang.xy
feat: support database
19ef0e3
xvyang.xy added 3 commits February 13, 2023 16:52
xvyang.xy
feat: add database & remove tenant
13a2313
xvyang.xy
feat: remove TablePointsBuilder
e4bf1bc
xvyang.xy
fix: fix problems in pr conversation
6bb1d34
MichaelLeeHZ
MichaelLeeHZ previously approved these changes Feb 13, 2023
View reviewed changes

@MichaelLeeHZ MichaelLeeHZ left a comment

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

LGTM

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@mrrtree
mrrtree dismissed MichaelLeeHZ’s stale review via 1b09742 February 13, 2023 11:59
@mrrtree
mrrtree force-pushed the feat/refactor_api branch from 1b09742 to c5a9bd8 Compare February 13, 2023 12:01
xvyang.xy
doc: rename metrics metric to table
7cf9835
@mrrtree
mrrtree force-pushed the feat/refactor_api branch from c5a9bd8 to 7cf9835 Compare February 13, 2023 12:35
ZuLiangWang
ZuLiangWang approved these changes Feb 14, 2023
View reviewed changes

@ZuLiangWang ZuLiangWang left a comment

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

LGTM

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@ZuLiangWang
ZuLiangWang merged commit c0df97c into apache:main Feb 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

2 more reviewers
@ZuLiangWang ZuLiangWang ZuLiangWang approved these changes
@MichaelLeeHZ MichaelLeeHZ MichaelLeeHZ left review comments
Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

4 participants

@mrrtree @CLAassistant @ZuLiangWang @MichaelLeeHZ
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.