sematext / sematext-agent-java

Sematext Monitoring Agent
https://sematext.com/spm
Apache License 2.0
23 stars 9 forks source link

SC-4121: Modify Java AA to use new naming #45

Closed sivasamyk closed 5 years ago

sivasamyk commented 5 years ago

Won't be merged to master till receiver changes are merged & deployed

Made the following changes:

1) SPM_MONITOR_TAGS in config properties to SPM_MONITOR_TAG_ALIASES 2) /write?db=tags endpoint to /write?db=tagAliases 3) tags_endpoint to tag_aliases_endpoint 4) All classes/functions & variables 5) tag name space to tag.alias 6) tags.type to tag.alias.type

1) Tested if the custom config is sent to the receiver (adjusted receiver accordingly) & persisted properly 2) Tested if migration of old ( 2.4.0 ) and new agent work properly by renaming the config property. 3) Tested if migration between 3.x versions work properly

sivasamyk commented 5 years ago

AFAIK Yes :) Verified by sending data to tag-receiver. Maybe when I do server-side changes, I might find some places that need changes.

otisg commented 5 years ago

Should we merge this now?

sivasamyk commented 5 years ago

I will merge all renaming related tasks tomorrow morning and deploy in prod.