Dev content review 5

[gkwan-ibm]

• Additional prerequisites
  • [x] better explain the application requires Kafka but it will use Kafka docker container instead to reduce the installation and configuration of Kafka
• What is asynchronous programming?
  • [x] the second paragraph, "your waiter" -> "your waitstaff"
  • [x] "... and their CPU usage statistics." I am not sure that phase because the inventory not storing the statistics.
  • the diagarm
  • [x] the middle graph including text should be bigger
  • [x] system and inventory boxes should be aligned by size and center
• MicroProfile Reactive Messaging Key Concepts
  • the diagarm
  • [x] Is @channel, @outgoing, @incoming diagarm better?
  • If use the exising,
    • [ ] the middle graph including text should be bigger
    • [ ] system and inventory boxes should be aligned by size and center
    • [ ] "cpuStatusTopic topic" vs "cpuStatusTopic"
• Building the Producer System microservice
  • [x] consider "Building Producer in the System microservice"
  • [x] don't use "We", "our"
  • [x] "...which is configured in the microprofile-config.properties.." -> "...which will configured in the microprofile-config.properties at the below section ..."
• Building the Consumer Inventory microservice
  • [x] consider "Building Consumer in the Inventory microservice"
  • [x] "...inventory/systems endpoint." add hotspot the inventory/systems
• [x] use SystemLoad vs CpuUsage
  • https://github.com/OpenLiberty/draft-guide-microprofile-reactive-messaging/blob/system-inventory/finish-get-property/models/src/main/java/io/openliberty/guides/models/SystemLoad.java
  • the services are using SystemLoad in finish-get-property but they handle multiple topics. You can reduce the code or refactor your existing code

[ankitagrawa]

Additional prerequisites better explain the application requires Kafka but it will use Kafka docker container instead to reduce the installation and configuration of Kafka

I am already explaining that in the What is Kafka Section. @gkwan-ibm