Skip to content

Thread local storage pattern #77

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.

Sign up for GitHub

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

Jump to bottom
Closed
iluwatar opened this issue Jun 23, 2015 · 8 comments
Closed

Thread local storage pattern #77

iluwatar opened this issue Jun 23, 2015 · 8 comments
Labels
epic: pattern
Milestone
1.15.0

Comments

@iluwatar
Copy link
Owner

https://en.wikipedia.org/wiki/Thread-local_storage
@thomasoss
Copy link
Contributor

I would like to deal with this
Greetings Thomas

@iluwatar
Copy link
Owner Author

Yes, great @thomasoss, the issue is yours!

@thomasoss
Copy link
Contributor

thomasoss commented Dec 13, 2016 •
edited
Loading

The coding is done and I also worked through the Checkstyle Pluging messages. But for using "ObjectAid UML Explorer for Eclipse" the purchase of a license is obligatory. Is there another way to create the UMLs in the format you need? Greetings Thomas

@mmena1
Copy link

mmena1 commented Dec 13, 2016 •
edited
Loading

@thomasoss The license is only needed if you want to use the extra diagrams, the free version already has the class diagram feature which is the one you need. When installing the plugin do not check the options with license requirements.

@thomasoss
Copy link
Contributor

@mrcrow85 thank you. Worked.
Now the UML is created. Looking into the existing etc folders is see xyz.urm.puml-files too. Can ObjectAid create this file. I couldn't find out how.
In the readme.md there is a pumlid: Where does this come from?
Greetings Thomas

@iluwatar
Copy link
Owner Author

We are in the process of switching the tool used to create the UML diagrams. This is discussed in #190. If you want, you can try the steps proposed by @markusmo3 to generate the pumlid or optionally just leave the pumlid empty for now.

@thomasoss
Copy link
Contributor

... ok - the tls.urm.puml was generated by the Maven build. Because my environment is configured for Gradle I had to do the Maven build in a second Eclipse project to which I copied the source code files. And I oversaw the puml created in the etc folder.

@thomasoss
Copy link
Contributor

@iluwatar and @markusmo3
creation of the pumlid worked fine with your instructions. Thank you

iluwatar added a commit that referenced this issue Apr 1, 2017
@iluwatar
Merge pull request #529 from thomasoss/master
854101b 
Thread Local Storage issue #77
@iluwatar iluwatar added this to the 1.15.0 milestone Apr 1, 2017
@iluwatar iluwatar closed this as completed Apr 1, 2017
iluwatar added a commit that referenced this issue Apr 1, 2017
@iluwatar
#77 Add missing license header
6ba4d7b
kemitix added a commit to kemitix/java-design-patterns that referenced this issue Apr 1, 2017
@kemitix
Merge remote-tracking branch 'upstream/master' into data-bus
30315e7 
* upstream/master: (67 commits)
  Set version for next development iteration
  Reached milestone 1.15.0
  iluwatar#539 Checkstyle fix
  iluwatar#539 More Checkstyle fixes
  iluwatar#539 Checkstyle fixes
  iluwatar#497 Add missing puml and license headers
  iluwatar#77 Add missing license header
  Work on iluwatar#190: urm/puml updates * added pumlid where it was missing and possible * removed pumlid where it generated a bad image * regenerated some incorrect puml's * added 'left to right direction' puml prefix to some diagrams to improve the automatic layouting
  Removed AvoidStarImport Rule Added JavaDocType Rule
  Update App.java
  Update App.java
  Update after changes from review. Additional improvements.
  Pom.xml files fixed.
  iluwatar#66 Balking Pattern
  fixed pmd violation
  fixed checkstyle violations
  diagrams added
  comments, tests and description
  changed parent POM version iluwatar#69
  apply changes from review iluwatar#69
  ...

# Conflicts:
#	pom.xml
pratigya0 pushed a commit to pratigya0/java-design-patterns that referenced this issue Aug 3, 2023
Merge pull request iluwatar#77 from noartem/master
b092de0 
Add golang questions about arrays, heap, sync
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
epic: pattern
Projects
None yet
Milestone
1.15.0
Development

No branches or pull requests
3 participants
@iluwatar @mmena1 @thomasoss