Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

Handle target rows case (Filter, Switch/Case, ...) #8

Open
mattcasters opened this issue Dec 7, 2018 · 13 comments
Open

Handle target rows case (Filter, Switch/Case, ...) #8

mattcasters opened this issue Dec 7, 2018 · 13 comments
Assignees
Labels
bug Something isn't working

Comments

@mattcasters
Copy link
Owner

No description provided.

mattcasters added a commit that referenced this issue Dec 7, 2018
mattcasters added a commit that referenced this issue Dec 7, 2018
mattcasters added a commit that referenced this issue Dec 7, 2018
@mattcasters
Copy link
Owner Author

Added test-cases for Filter Rows and Switch/Case, the most popular ones.
I have now confidence the generics work now behind this.

mattcasters added a commit that referenced this issue Dec 16, 2018
@twknight
Copy link

This appears to be broken for me (using latest 0.6/Master versions). With the Filter Rows step, I can only get output using the Main Output from the step. If I direct any rows (True or False), no rows are output (even with one still using Main Output). See the following:
image
image
image
image

It seems like an issue with creating the additional PCollection outputs, as the graph shows the same tag ("out0") for both targets' input step. I'm not sure why creating one target (a False result in my example) destroys the records for the step's Main Output, but that's what I'm observing.

@mattcasters
Copy link
Owner Author

To be fair, that filter step is not configured correctly (otherwise said, something I have never seen, didn't know it was working :-))

So either just filter the rows with a default output or user a True/False split.

@mattcasters mattcasters self-assigned this Jun 12, 2019
@mattcasters mattcasters added the bug Something isn't working label Jun 12, 2019
@twknight
Copy link

Sorry, I didn't include the most obvious case in my example, but a True/False split isn't working either, which is what led me to try the Main Output. I've completely removed the step, saved, tried again...same results each time.

@mattcasters
Copy link
Owner Author

Should be easy to fix.

@mattcasters
Copy link
Owner Author

Is filter-split-combine.ktr working for you?

@mattcasters mattcasters reopened this Jun 12, 2019
@mattcasters
Copy link
Owner Author

Also, this is an old case, it got me confused for a while.
If you have bugs, please simply report them, the more the better, TIA :-)

@mattcasters
Copy link
Owner Author

Well, FWIW, both the split and the filter-default.ktr example (I just added it to kettle-beam-examples) works fine. Try to update to kettle-beam v1.0.0 and recompile it.

@twknight
Copy link

filter-split-combine.ktr is working correctly. I also modified it to filter on the integer id column, which also worked. Only obvious difference I'm seeing is the BigQuery source in mine, but hard to imagine that Kettle sees the records any differently when they arrive to the step.

I'll get v1.0.0 and try again. Thanks for your help.

@twknight
Copy link

It appears the TRUE/FALSE targeted output only works if going to another step prior to Beam Output. Simply adding a Dummy step between the Filter and the outputs allowed it to work. Perhaps the grouping/flattening taking place in Beam Output is hard-coded to the default PCollection?
image

@mattcasters
Copy link
Owner Author

Sorry, I can't confirm this. Works just fine on my end.
I created test-transformation "filter-direct-output.ktr" in the kettle-beam-examples project

@virivigio
Copy link

Actually it still happens on kettle-neo4j-remix-beam-8.2.0.7-719-REMIX.zip
"filter-direct-output.ktr" is affected as well, just remove the "Add constants" steps

@mattcasters
Copy link
Owner Author

Urgh, you're right. It regressed somewhere along the line. The pipeline is created correctly but the data passing isn't happening.

image

To catch it you would have to look at the metrics.

image

@mattcasters mattcasters reopened this Dec 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants