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

WIP test: add drilldowns[LABEL].columns[NAME].stage test case without cache #680

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kenhys
Copy link
Member

@kenhys kenhys commented Apr 24, 2017

When the value of --drilldowns[LABEL].columns[NAME].stage parameter is
changed, Groonga returns result without using cache.

… cache

When the value of --drilldowns[LABEL].columns[NAME].stage parameter is
changed, Groonga returns result without using cache.
@kenhys
Copy link
Member Author

kenhys commented Apr 24, 2017

I've expected that output of this case is different, but not. any idea?

@kou
Copy link
Member

kou commented Apr 24, 2017

Could you try --drilldowns[LABEL].filter instead of --query?

@kenhys
Copy link
Member Author

kenhys commented Apr 24, 2017

Could you try --drilldowns[LABEL].filter instead of --query?

Could you explain what you mean precisely? @kou
In other test case, we discussed that --filter is not worked expectedly for stage, so switched to use --query.
In this case, I think that the situation is same as before. But is --drilldowns[LABEL].filter effective?

@kou
Copy link
Member

kou commented Apr 24, 2017

Issue number please.

@kenhys
Copy link
Member Author

kenhys commented Apr 24, 2017

In other test case

I means #678 as other test case.

@kou
Copy link
Member

kou commented Apr 24, 2017

It's a --columns[NAME] test. This is a --drilldowns[LABEL].columns[NAME] test.

@kenhys
Copy link
Member Author

kenhys commented Apr 25, 2017

Note that filtered stage is not supported yet. so this test always fails.
And the following part is meaningless because query is already processed before drilldown.

--match_columns items \
+  --output_columns content \
+  --query "Book"

See http://groonga.org/ja/docs/reference/commands/select.html#select-columns-name-stage about processing order of dynamic columns and drilldowns.

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

Successfully merging this pull request may close these issues.

None yet

2 participants