Skip to content

Conversation

guidomedina
Copy link

Small changes:

  • Synchronization of time in millis removed.
  • Regex compiled pattern made static.
  • Slf4j updated to 1.7.18

@guidomedina guidomedina force-pushed the gm-static-pattern-unsync-time-and-slf4j-update branch from f5e839f to 9ea1a1d Compare February 28, 2016 14:32
@chrjohn
Copy link
Member

chrjohn commented Feb 29, 2016

This is for QFJ-879, right?

@guidomedina
Copy link
Author

Yeah, but decided to also update few minor things in a single commit. How do I add a comment to also affect JIRA automatically? is that configured?

@chrjohn chrjohn changed the title Small Changes: Static compiled pattern, un-sync time and slf4j update Small Changes: Static compiled pattern, un-sync time (QFJ-879) and slf4j update Feb 29, 2016
@chrjohn
Copy link
Member

chrjohn commented Feb 29, 2016

If you include the issue number in the title, JIRA has a link to the pull request on the right hand side. But AFAIK you cannot make github comments appear automatically in JIRA. Or maybe you need a special plugin for that.

@guidomedina guidomedina changed the title Small Changes: Static compiled pattern, un-sync time (QFJ-879) and slf4j update QFJ-879: Small Changes: Static compiled pattern, un-sync time and slf4j update Feb 29, 2016
@guidomedina guidomedina force-pushed the gm-static-pattern-unsync-time-and-slf4j-update branch 6 times, most recently from b134178 to 18194a6 Compare March 3, 2016 20:00
@guidomedina
Copy link
Author

Re-pushed with volatile, screw it, volatile will keep it correct and thread safe as writes are visible as soon as they are done, but TBH it is still slower than no volatile at all, though without it there is a risk of not seeing the value at all if you change the time source after some other thread have started.

At least volatile won't block concurrent sessions from getting the current time.

@guidomedina guidomedina force-pushed the gm-static-pattern-unsync-time-and-slf4j-update branch from 18194a6 to af890aa Compare March 3, 2016 20:11
Static final pattern variable names fixes.
Slf4j updated to 1.7.18
@guidomedina guidomedina force-pushed the gm-static-pattern-unsync-time-and-slf4j-update branch from af890aa to b5fe194 Compare March 3, 2016 20:18
chrjohn added a commit that referenced this pull request Mar 4, 2016
…-and-slf4j-update

QFJ-879: Small Changes: Static compiled pattern, un-sync time and slf4j update
@chrjohn chrjohn merged commit 8e13e44 into quickfix-j:master Mar 4, 2016
@guidomedina guidomedina deleted the gm-static-pattern-unsync-time-and-slf4j-update branch March 4, 2016 22:15
@chrjohn
Copy link
Member

chrjohn commented Mar 4, 2016

OK, thank you :)

chrjohn added a commit that referenced this pull request Mar 4, 2016
…-and-slf4j-update

QFJ-879: Small Changes: Static compiled pattern, un-sync time and slf4j update
Conflicts:
	pom.xml
@chrjohn chrjohn added this to the QFJ 1.6.2 milestone Mar 24, 2016
david-gibbs-ig added a commit to IG-Group/quickfixj that referenced this pull request Apr 21, 2020
…4-4 to QFJ_RELEASE_2_1_0_IG

* commit '371329943fa6f3b9b1ae1ba32b0f3a912333288a':
  FIXAPI-1808 - Added missing tag
  FIXAPI-1808 - Added Strategy group to 4.4
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

Successfully merging this pull request may close these issues.

2 participants