-
Notifications
You must be signed in to change notification settings - Fork 0
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
Outputs #1
Comments
I think an output for each entry of an RSS works best with the pubsubhubbub RSS Trigger, whereas this is a RSS Action. Tom - did I understand your suggestion correctly? Jeff - can you weigh in here? |
This isn't really an action, is it? Combined with a "cron" trigger it would be a polling RSS trigger, so I'd expect it to have the same outputs as a pubsubhubub RSS trigger. |
I think it's firmly in the action camp. You could combine it with the cron On Tue, Nov 27, 2012 at 5:14 PM, Tom Robinson [email protected]:
V/R Matthew Hudson |
Yeah, I think most output names should be singular unless you really are On Tue, Nov 27, 2012 at 2:26 PM, Matthew Hudson [email protected]:
Jeff Lindsay |
Well, if the two options are "trigger" and "action" then it's an action, but I'd rather just call it a block since it has outputs. But that's irrelevant. The way I see the "dedup" block working is it would have a "key" input used for identifying duplicates, and a "" input you could use to passthrough the rest of the fields, which would be mirrored as outputs (so maybe we need a "" output as well?). You'd connect the individual RSS field outputs from the RSS parse block to the dedup block, then each output to the next block. Granted this approach will be tedious for records with many fields, but if you just connect an "entry" output then the field names can't be propagated to "downstream" blocks. Also I'm still a bit confused and concerned about how blocks that output multiple records will work. Say you have two blocks that can output multiple records connected to two different inputs on a 3rd block, how does the executor "merge" the records? |
In theory it might have a lot to do with the dependency resolution and Jeff Lindsay |
The problem is I can't think of a concrete example, but right now there's Maybe restricting a pipeline to having a single trigger or other block that On Tue, Nov 27, 2012 at 3:44 PM, Jeff Lindsay [email protected]:
|
Yeah I was already under the impression there's only one pure trigger. On Tue, Nov 27, 2012 at 3:53 PM, Tom Robinson [email protected]:
Jeff Lindsay |
But allowing any block to output multiple records is also an issue. |
Right, because it effectively becomes a trigger. Hmm.. On Tue, Nov 27, 2012 at 4:04 PM, Tom Robinson [email protected]:
Jeff Lindsay |
I think there should be an output for each property of an RSS entry (title, description, url, etc), rather than just one called "entries" (also all output names should be singular, right?)
The text was updated successfully, but these errors were encountered: