apache / incubator-horaedb-client-java Goto Github PK

Describe this problem

Steps to reproduce

1. Create table

curl --location --request POST 'http://127.0.0.1:5440/sql' --data-raw '
CREATE TABLE `demo` (
    `name` string TAG,
    `value` double NOT NULL,
    `t` timestamp NOT NULL,
    `t2` timestamp NOT NULL,
    timestamp KEY (t))
ENGINE=Analytic
  with
(enable_ttl="false")

2. Insert a row

curl --location --request POST 'http://127.0.0.1:5440/sql' --data-raw '
INSERT INTO demo (t, t2, name, value)
    VALUES (1651737067000, 1651737067009, "ceresdb", 100)
'

3. Query by http interface

curl --location --request POST 'http://127.0.0.1:5440/sql' --data-raw '
SELECT t, avg(value) FROM `demo` group by t
'

response:

{"rows":[{"t":1651737067000,"AVG(demo.value)":100.0}]}

4. Query by Java SDK

        CeresDBClient client = getClient();
        String query = "SELECT t, avg(value) FROM `demo` group by t";
        SqlQueryRequest request = new SqlQueryRequest(query);
        CompletableFuture<Result<SqlQueryOk, Err>> future = client.sqlQuery(request);
        Result<SqlQueryOk, Err> result = future.get();
        System.out.println(result);

The response's rowCount is 0.

Expected behavior

Additional Information

Description

Add CI with Github Actions

Proposal

Let me send a PR for this

Additional context

Details: package name of the java code still uses "com.ceresdb", but the project official site is "io.ceresdb".

Describe this problem

In parseArrowBatch, it will return an empty list when there is io exception, which is totally wrong.

https://github.com/CeresDB/ceresdb-client-java/blob/3f5a19f724e26dfb2b1a98053d64230689aef80f/ceresdb-protocol/src/main/java/io/ceresdb/util/Utils.java#L465

Steps to reproduce

No need.

Expected behavior

It should return the exception to upper layer.

Additional Information

add issue template

• feature
• bug
• question

IDE will generate local files in the project dir, should be ignored

Describe this problem

ceresdb server is running as single mode, image: ceresdb/ceresdb-server: nightly-20230223-ce9fa87

Report NPE for table error while creating table

Steps to reproduce

Expected behavior

Additional Information

Description
When the routing information of the ceresdb partition table is empty, the java sdk panics.

Proposal
Avoid panic when endpoint in route is null.
https://github.com/CeresDB/ceresdb-client-java/blob/9f9da9564f853c4d0baf413b5bff5d0ff00778aa/ceresdb-protocol/src/main/java/io/ceresdb/RouterClient.java#L412

Additional context

Description

The proto files and generated code should be managed in a single repo, rather than scatter them everywhere. It will be more maintainable that other projects depend on one single proto repo.

Proposal

Since ceresdb-client-java is a pure Java written project, we can add ceresdb-proto-internal as a dependency to the pom. And then remove the protos and related codes maintained inside this project.

Additional context

no

Describe this problem

When there are lambda expressions in the code files, code formatter does not work.

Steps to reproduce

use maven -X to see the debug log

Expected behavior

All java codes, except the generated files should be formatted.

Additional Information

Describe this problem
Create table and Drop table need to use routing.
But the current logic is using the management addr to create table.

Steps to reproduce

Expected behavior

Additional Information

Describe this problem
Using the image of ceresdb:v1.0.0 and the sdk of version 1.0.1, when the query data contains Chinese characters, some servers have garbled characters

Steps to reproduce
By setting JAVA_OPTS="$JAVA_OPTS -Dfile.encoding=US-ASCII"

Expected behavior
Chinese characters are not garbled

Additional Information

Describe this problem

It occurs Table not found when aliasing a table name in sql

Steps to reproduce

Run this sql with sdk

select * from `SPM_2197803522_INFLUENCE_DEFAULT` t1 
WHERE 
t1.`period` between '2023-03-22 11:09:00' and  '2023-03-22 11:09:59'
limit 10

Expected behavior

No error happened

Additional Information

Description
CeresDB grpc service has already supported CreateTable and DropTable.

Proposal
Remove the http service setting, CreateTable and DropTable uses grpc service.

Additional context

Description
Current client APIs are stable and have passed many tests in different scenarios. It's time to make a first formal release.

Proposal

Release 0.1.0

Additional context
No.

Eclipse will produce some config file, should be ignored

Eclipse m2e errors, such as:

• Plugin execution not covered by lifecycle configuration: com.googlecode.maven-java-formatter-plugin:maven-java-formatter-plugin:0.4:format (execution: default, phase: process-sources)

Description

I found table routeCache will not refresh when routeCache size is too small.

   if (this.routeCache.size() < this.opts.getMaxCachedSize() * CLEAN_CACHE_THRESHOLD) {
            LOG.info("Now that the number of cached entries is {}.", this.routeCache.size());
            return times;
        }

The problem is:
Request will be routed to wrong server after table is recreated.

Proposal

Support refresh table routing info in Client periodly

Additional context

Describe this problem

Memory leak for sdk 1.0.0

Steps to reproduce

Expected behavior

Additional Information