Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Aggregate function sum() not supported on a varint or decimal column #145

Closed
rkarthik007 opened this issue Apr 7, 2018 · 1 comment
Closed
Assignees
Labels
kind/enhancement This is an enhancement of an existing feature

Comments

@rkarthik007
Copy link
Collaborator

Test case:

cqlsh> create keyspace if not exists k;
cqlsh> use k;
cqlsh:k> create table t (k int primary key, n decimal);
cqlsh:k> insert into t (k, n) values (1, 1);
cqlsh:k> insert into t (k, n) values (2, 2);
cqlsh:k> select * from t;

 k | n
---+---
 1 | 1
 2 | 2

(2 rows)
cqlsh:k> select sum(n) from t;
InvalidRequest: Error from server: code=2200 [Invalid query] message="SQL error (yb/yql/cql/ql/ptree/process_context.cc:51): Invalid Function Call. Failed calling 'sum(decimal)'. Not implemented (yb/util/bfql/bfql.cc:228): Builtin function 'sum' is not yet implemented
select sum(n) from t;
       ^^^
 (error -214)"

Reported by @robertpang

@rkarthik007 rkarthik007 added the kind/bug This issue is a bug label Apr 7, 2018
@rven1 rven1 assigned rven1 and unassigned m-iancu Jun 28, 2018
yugabyte-ci pushed a commit that referenced this issue Jul 9, 2018
Summary:
This diff implements support for sum for varint and decimal columns.
Tests TestDecimalDataType.java and TestVarIntDataType.java have been
enhanced to test for sum of the given types.

Test Plan: TestDecimalDataType.java and TestVarIntDataType.java

Reviewers: mihnea, robert, neil, hector

Reviewed By: hector

Subscribers: kannan, yql

Differential Revision: https://phabricator.dev.yugabyte.com/D5065
@kmuthukk kmuthukk added kind/enhancement This is an enhancement of an existing feature and removed kind/bug This issue is a bug labels Jul 9, 2018
@rven1
Copy link
Contributor

rven1 commented Jul 11, 2018

@rven1 rven1 closed this as completed Jul 11, 2018
tvesely pushed a commit to tvesely/yugabyte-db that referenced this issue Feb 28, 2024
… protocol

A query executed with the extended query protocol may entirely ignore
hints defined in it, as the only moment where a hint is reset is when a
query is restarted.  This commit introduces a hook for ExecutorEnd(),
which is a code path taken at the end of each individual execution
happening through a Portal when using the extended query protocol.

After this patch, it should be possible to make the reset of
current_hint_retrieved less optimistic, but the current logic does not
hurt as well as the end of executor would match with the next query to
come in for most users.

Per report and patch yugabyte#12741 in yugabyte-db, also reported as yugabyte#145 in
pg_hint_plan.

Reported-by: Furutani-san (kfuru)
Author: tanujnay112
Backpatch-through: 11
jasonyb pushed a commit that referenced this issue Mar 18, 2024
…number

Fix version number in install script.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement This is an enhancement of an existing feature
Projects
None yet
Development

No branches or pull requests

4 participants