Skip to content
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

HornetQ to Artemis migration #9

Closed
c0c0n3 opened this issue Jun 16, 2017 · 3 comments
Closed

HornetQ to Artemis migration #9

c0c0n3 opened this issue Jun 16, 2017 · 3 comments

Comments

@c0c0n3
Copy link
Owner

c0c0n3 commented Jun 16, 2017

HornetQ was donated to Artemis, so we have to migrate to Artemis. Also, the SpingBoot HornetQ starter reached end of life in the 1.4.x series and is not available for 1.5.x.

Purely from a code migration standpoint, It doesn't look like this is a train smash for us. In fact,

  • Artemis still has the same classes of the HornetQ core API we've used, only in a different namespace
  • SpringBoot should work exactly the same according to the 1.3.4 docs

But is the HornetQ core functionality we've used still the same in Artemis? Only time and lots of testing will tell...

This was referenced Jun 16, 2017
@c0c0n3
Copy link
Owner Author

c0c0n3 commented Jun 22, 2017

Docs need an upgrade too! (Warning: donkey work ahead...)

Here's the list of docs site sections to update:

  • Configuration / Security / File System
  • Design / Tech Stack
  • High-level View / Conceptual Overview (including diagram)
  • High-level View / Codebase Essentials (including diagram)
  • Messaging (including diagrams: key-q-classes.svg, omero-import-gc-wiring.svg, omero-import-wiring.svg)
  • Process View / Processes & Threads
  • Process View / Distribution

And for the JavaDoc, all the HornetQ classes in:

  • ome.smuggler.config
  • ome.smuggler.providers.q

@c0c0n3 c0c0n3 mentioned this issue Jun 22, 2017
9 tasks
@c0c0n3
Copy link
Owner Author

c0c0n3 commented Jun 26, 2017

Dead letter/expire address warnings

Artemis logs warning messages at start up for each and every queue that has no associated dead letter/expiry address. (See this issue.) We're not using dead letters and expiry addresses at the moment cos we have our own failure handling mechanism built into the messaging API. Couldn't find an easy way to tell Artemis to suppress those warnings, so for the time being we'll have to live with them.

c0c0n3 added a commit that referenced this issue Jun 27, 2017
this commit only changes the source code to make it compile with the new
Artemis and SpringBoot/Artemis libs. There are some critical features of
HornetQ we rely on, so we have to check Artemis still works the same, see
TODOs in ServerConnector and DequeueTask.
@c0c0n3
Copy link
Owner Author

c0c0n3 commented Jun 27, 2017

implemented changes in v1.1.0-beta, see #11.

@c0c0n3 c0c0n3 closed this as completed Jun 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant