-
Notifications
You must be signed in to change notification settings - Fork 388
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
CLDR-14316 doc: document SurveyTool setup #3135
Conversation
wip… |
see also: https://docs.google.com/document/d/1wkJsoQXPcGN_dlCzXzwhfjQoq9Mes7VezIRmNrHWV8c/edit I just added the following sentence to that doc, which otherwise hasn't been edited for a long time: 'Note: Tom has switched from Eclipse to IntelliJ in recent months, and debugs ST by starting it in Terminal, then choosing “Attach to Process” from the “Run” menu in IntelliJ.' |
@btangmu I'm planning to leave this open until after the merge (no reason to put it into 44) |
@btangmu I'd like to merge this in v45 so moving out of draft |
603db6f
to
db33d48
Compare
Hooray! The files in the branch are the same across the force-push. 😃 ~ Your Friendly Jira-GitHub PR Checker Bot |
db33d48
to
d898a09
Compare
Notice: the branch changed across the force-push!
~ Your Friendly Jira-GitHub PR Checker Bot |
@btangmu PTAL. If this is an improvement let's merge it otherwise let me know what you'd like to see. You had asked about logging for example. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM -- FYI my own local setup has "root" instead of "surveytool" for the mysql user -- the username doesn't matter as long as you're consistent
also, I always import an existing db from a dump/backup rather than creating a new db, and that might be worthwhile to suggest as an alternative in this doc
My server.env also has these two lines:
WLP_DEBUG_SUSPEND=n
WLP_DEBUG_ADDRESS=7777
-- I'm not sure if they're needed, and I don't remember when/why I added them
A link to this doc might also be helpful here, even though it's also outdated: https://docs.google.com/document/d/1wkJsoQXPcGN_dlCzXzwhfjQoq9Mes7VezIRmNrHWV8c/edit#heading=h.ozmz4163qnxu
you can run it as root, but i wouldn't recommend it. particularly, we might miss if a SQL action requires elevated privileges. some of the forum report code manipulates stored procedures for example. is that document owned by the U+CLDR-TC drive? If not would you consider moving it there? I don't know what the suspend is, but https://www.ibm.com/docs/en/was-liberty/core?topic=liberty-customizing-environment says that 7777 is the default port. |
CLDR-14213
ALLOW_MANY_COMMITS=true