-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Project Structure - Reorganize in favor of JavaDSL and Reactive support #319
Comments
I probably agree with you and we really need to dedicate some time to improve this repo. The Loanshark is a part of well-known Loan Broker EIP use-case: https://www.enterpriseintegrationpatterns.com/SystemManagementExample.html |
@artembilan That's awesome! Sorry for not doing the full homework before the post. I've edited the suggested project structure for your review. If you approve, I can create a PR for the implementation. |
@artembilan What do you think? |
I'm not ready to answer yet, but I disagree for We may just move away from those confusing There are might be some new samples like Still need to come up with some working plan... |
I think separation like Another alternative might be to specify for each example |
Yeah... All of these just sound like your own preferences. It doesn't mean that the way you see it is going to helpful for other people. It is great that we have such a conversion, but right now it is just my opinion against yours. |
@artembilan you really prefer all of the samples to be at the same level? This will be pretty hard to view |
In my feeling that is going to be the more intuitive and neutral way. In most cases people are looking for some specific protocol or use cases like many other ideas? |
@markusherbert I have the same confusion when looking at these examples, especially in the beginning, it is too difficult to understand a complete process. The cost of learning is high. I also wish I had a |
@artembilan Are there any plans for introducing Java DSL based examples in the near future? |
I cannot promise anything since we are here short-handed and have other priorities. |
Thanks for the update @artembilan |
Currently, this project looks like a project that has lots of examples. Underlying these examples are mostly XML-based, which is super unreadable for new users. I think this project should be reorganized and instead of dirs like
intermediate
,applications
,advanced
,basic
,dsl
, there's should be:xml
basic
intermediate
advanced
java
reactive
basic
intermediate
advanced
non-reactive
basic
intermediate
advanced
specific-applications
xml
java
reactive
non-reactive
Notice that if a sample is JavaDSL based, it is super unreadable to use inside of its parts of another XML-based POJOs.
Another horrible thing is that there are samples whose names are not based on the libraries, but some random names like "Loanshark". The purpose of samples is to make it clear how the usage looks. The application purpose is irrelevant and confusing.
I don't know what is
applications
place in the world. I'm also not sure ifbasic
,intermediate
, andadvanced
separation is needed. People would be fine if there would all sit together.This really doesn't require a lot of work and can be super helpful to people. Even if you open an empty folder for
reactive
, or very thin folders forjava
in general, it would let people comfortably contribute.The text was updated successfully, but these errors were encountered: