You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@streampipes.apache.org by "bossenti (via GitHub)" <gi...@apache.org> on 2023/02/19 15:40:10 UTC
[I] [WIP] Migrate processing elements from three class to once class approach (streampipes)
bossenti opened a new issue, #1315:
URL: https://github.com/apache/streampipes/issues/1315
### Body
Historically, we used a three class based approach to define our pipeline elements (processors and sinks).
A while ago we introduced a simplified one class approach to define pipeline elements.
Therefore, we need to migrate all existing pipeline elements.
### Processors
Old approach: A processor is defined by the following three classes:
Examples can be found below.
New approach: Only use the following class to define a processing element:
Processors to migrate:
-
### StreamPipes Committer
I acknowledge that I am a maintainer/committer of the Apache StreamPipes project.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@streampipes.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Migrate processing elements from three class to once class approach (streampipes)
Posted by "bossenti (via GitHub)" <gi...@apache.org>.
bossenti commented on issue #1315:
URL: https://github.com/apache/streampipes/issues/1315#issuecomment-1437416945
@BudhirajaChinmay great to hear 🎉
I have created a sub-task for you: https://github.com/apache/streampipes/issues/1330
If you comment there anything (e.g., "assign me"), I should be able to assign you to the issue.
Please let us know, if you need any help :)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@streampipes.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Migrate processing elements from three class to once class approach (streampipes)
Posted by "flaxxen (via GitHub)" <gi...@apache.org>.
flaxxen commented on issue #1315:
URL: https://github.com/apache/streampipes/issues/1315#issuecomment-1448374646
Hello! Me and some friends have a school project to work with a refactor issue this week and would like to work on the boolean counter migration.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@streampipes.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Migrate processing elements from three class to once class approach (streampipes)
Posted by "bossenti (via GitHub)" <gi...@apache.org>.
bossenti commented on issue #1315:
URL: https://github.com/apache/streampipes/issues/1315#issuecomment-1478520458
Awesome, here you go :)
https://github.com/apache/streampipes/issues/1434
https://github.com/apache/streampipes/issues/1436
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@streampipes.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Migrate processing elements from three class to once class approach (streampipes)
Posted by "BudhirajaChinmay (via GitHub)" <gi...@apache.org>.
BudhirajaChinmay commented on issue #1315:
URL: https://github.com/apache/streampipes/issues/1315#issuecomment-1437509868
@bossenti according to my understanding, the processors in the stream-apis-processor-image-jvm (Image Enricher, Image Cropper, GenericImageClassification and QrCodeReader) would be interlinked for depreciating the old flow as they are using a particular set of common helper classes. I can work on these together. If this approach seems ok to you, you can assign these four processors to me.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@streampipes.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Migrate processing elements from three class to once class approach (streampipes)
Posted by "bossenti (via GitHub)" <gi...@apache.org>.
bossenti commented on issue #1315:
URL: https://github.com/apache/streampipes/issues/1315#issuecomment-1449687180
@flaxxen Great to hear 🤩
I have created an issue for you: https://github.com/apache/streampipes/issues/1366
If you give it a comment, I can assign you to it 🙂
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@streampipes.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Migrate processing elements from three class to once class approach (streampipes)
Posted by "bossenti (via GitHub)" <gi...@apache.org>.
bossenti commented on issue #1315:
URL: https://github.com/apache/streampipes/issues/1315#issuecomment-1437575240
Sounds legit 👍🏼
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@streampipes.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Migrate processing elements from three class to once class approach (streampipes)
Posted by "tenthe (via GitHub)" <gi...@apache.org>.
tenthe commented on issue #1315:
URL: https://github.com/apache/streampipes/issues/1315#issuecomment-1484513075
Hi @liuxiaocs7,
I created #1457 & #1456.
Please comment there so I can assign you the issue.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@streampipes.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Migrate processing elements from three class to once class approach
Posted by "bossenti (via GitHub)" <gi...@apache.org>.
bossenti closed issue #1315: Migrate processing elements from three class to once class approach
URL: https://github.com/apache/streampipes/issues/1315
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@streampipes.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Migrate processing elements from three class to once class approach (streampipes)
Posted by "liuxiaocs7 (via GitHub)" <gi...@apache.org>.
liuxiaocs7 commented on issue #1315:
URL: https://github.com/apache/streampipes/issues/1315#issuecomment-1474899815
@bossenti I want to complete this task to be familiar with our pipeline elements, please assign StringCounter to me first.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@streampipes.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Migrate processing elements from three class to once class approach (streampipes)
Posted by "liuxiaocs7 (via GitHub)" <gi...@apache.org>.
liuxiaocs7 commented on issue #1315:
URL: https://github.com/apache/streampipes/issues/1315#issuecomment-1483874811
Hi, @bossenti, `StringToState` and `StringTimer` PRs has created, could you assign `KafkaPublisher` and `Redis` sink to me, I want to learn about the sink component, thanks! :)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@streampipes.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Migrate processing elements from three class to once class approach (streampipes)
Posted by "flaxxen (via GitHub)" <gi...@apache.org>.
flaxxen commented on issue #1315:
URL: https://github.com/apache/streampipes/issues/1315#issuecomment-1454850269
Hi again! Needed another task to fill out the time needed for the week, my group were a bit ahead of me writing so we have already mostly finished the boolean inverter class aswell, could you create another issue for that and we can provide a separate PR?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@streampipes.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Migrate processing elements from three class to once class approach (streampipes)
Posted by "tenthe (via GitHub)" <gi...@apache.org>.
tenthe commented on issue #1315:
URL: https://github.com/apache/streampipes/issues/1315#issuecomment-1449683246
Great, thank you! 🚀
Please provide a PR and we can discuss the changes there.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@streampipes.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Migrate processing elements from three class to once class approach (streampipes)
Posted by "bossenti (via GitHub)" <gi...@apache.org>.
bossenti commented on issue #1315:
URL: https://github.com/apache/streampipes/issues/1315#issuecomment-1474907903
@liuxiaocs7 Here you go: https://github.com/apache/streampipes/issues/1426
Please comment there, so I can assign you
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@streampipes.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Migrate processing elements from three class to once class approach (streampipes)
Posted by "BudhirajaChinmay (via GitHub)" <gi...@apache.org>.
BudhirajaChinmay commented on issue #1315:
URL: https://github.com/apache/streampipes/issues/1315#issuecomment-1437407382
@bossenti I would like to work on this issue. I am planning to start with KafkaPublisher.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@streampipes.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Migrate processing elements from three class to once class approach (streampipes)
Posted by "liuxiaocs7 (via GitHub)" <gi...@apache.org>.
liuxiaocs7 commented on issue #1315:
URL: https://github.com/apache/streampipes/issues/1315#issuecomment-1478336605
Hi, @bossenti, i want to work on `StringToState` and `StringTimer` this week, could you help me open issues separately, I will take assign in these two issues separately, thanks! :)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@streampipes.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Migrate processing elements from three class to once class approach (streampipes)
Posted by "bossenti (via GitHub)" <gi...@apache.org>.
bossenti commented on issue #1315:
URL: https://github.com/apache/streampipes/issues/1315#issuecomment-1455055470
@flaxxen awesome 🚀
Here you go: https://github.com/apache/streampipes/issues/1387
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@streampipes.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Migrate processing elements from three class to once class approach
Posted by "bossenti (via GitHub)" <gi...@apache.org>.
bossenti commented on issue #1315:
URL: https://github.com/apache/streampipes/issues/1315#issuecomment-2042313640
All pipeline elements are migrated
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@streampipes.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org