fix 'filter with empty name' error

[ricardopintottrdata]

Type of Change (select one):

• Bug fix

Description of the Change:
Fixes the error "filter with empty name"

Related Issue (provide the link):
#887

How to test:

mysql> drop table if exists t; create table t(a int, b int); insert into t values(0,1,2); select a, count(*) from t where a=1 or b=2 group by a having count(*) = 1;  
--------------  
drop table if exists t  
--------------  
Query OK, 0 rows affected (0.10 sec)  
  
--------------  
create table t(a int, b int)  
--------------  
Query OK, 0 rows affected (0.05 sec)  
  
--------------  
insert into t values(0,1,2)  
--------------  
Query OK, 1 row affected (0.00 sec)  
  
--------------  
select a, count(*) from t where a=1 or b=2 group by a having count(*) = 1  
--------------  
  
ERROR 1064 (42000): index t: filter with empty name  

Replacing or with and solves the problem:

select a, count(*) from t where a=1 and b=2 group by a having count(*) = 1  
--------------  
  
 ------ ----------   
| a    | count(*) |  
 ------ ----------   
|    1 |        1 |  
 ------ ----------   
1 row in set (0.01 sec)  

Removing HAVING also solves the problem:

select a, count(*) from t where a=1 or b=2 group by a  
--------------  
  
 ------ ----------   
| a    | count(*) |  
 ------ ----------   
|    1 |        1 |  
 ------ ----------   
1 row in set (0.00 sec)  

[ricardopintottrdata]

@sanikolaev hello, can you check this please ? the failing test was again the error that has nothing to do with the changes submitted. Thank you !

[sanikolaev]

Hello @ricardopintottrdata
These tests

are failing because:

• GitHub runs the CI in the context of your organization, not https://github.com/manticoresoftware
• the secret variables JINA_API_KEY, OPENAI_API_KEY, and VOYAGE_API_KEY are not defined in your organization or your repository

As said in the Github docs :

workflows from forks do not have access to sensitive data such as secrets

You can add these secrets, and that should stop the failures. We can also skip the tests if the variables are not set (implemented in this PR #3835), but I'm not sure it's a good solution. I'll check with my colleagues.

[sanikolaev]

@glookka please review the PR