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

Test fail #51

Open
Joezczhang opened this issue Oct 20, 2021 · 0 comments
Open

Test fail #51

Joezczhang opened this issue Oct 20, 2021 · 0 comments

Comments

@Joezczhang
Copy link

Joezczhang commented Oct 20, 2021

Hello:

when I test this extension with command make installcheck,ERROR named desired number of counters is higher than the topn.number_of_counters variable happened, just like blow:

test add_agg ... ok 427 ms
test union_agg ... ok 235 ms
test char_tests ... FAILED 44 ms
test null_tests ... ok 23 ms
test add_union_tests ... FAILED 77 ms
test copy_data ... ok 2254 ms
test customer_reviews_query ... ok 1840 ms
test join_tests ... FAILED 819 ms

information in char_tests.out:
SELECT (topn(topn_union_agg(jsonb_column), 9)).*
FROM jsonb_table;

  •            item                 | frequency 
    

--------------------------------------+-----------

  • TR | 6
  • CH | 4
  • US | 4
  • FRA | 2
  • ' | 1
  • '"" +++--- <>?//#$%^&*()_+!@/t | 1
  • \\ | 1
  • '"" | 1
  • """"" | 1
    -(9 rows)

+ERROR: desired number of counters is higher than the topn.number_of_counters variable
do
$$
declare
@@ -91,19 +79,7 @@
FROM values_table;
SELECT (topn(topn_union_agg(jsonb_column), 9)).*
FROM jsonb_table;

  • item     | frequency 
    

---------------+-----------

  • кйльжзхцвбнм | 35
  • эасдфгч | 30
  • тыуио | 20
  • пющ | 12
  • ёъяшер | 10
  • TR | 9
  • CH | 6
  • US | 6
  • FRA | 3
    -(9 rows)

+ERROR: desired number of counters is higher than the topn.number_of_counters variable
do
$$
declare

I find this error in function topn in topn.c,what confuses me is why SRF_IS_FIRSTCALL() is always true.

Hope for your help.

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

No branches or pull requests

1 participant