Skip to content

MINOR: Updated Quickstart to mention log.dirs#5361

Merged
harshach merged 1 commit intoapache:trunkfrom
mimaison:quickstart_fix
Jul 18, 2018
Merged

MINOR: Updated Quickstart to mention log.dirs#5361
harshach merged 1 commit intoapache:trunkfrom
mimaison:quickstart_fix

Conversation

@mimaison
Copy link
Copy Markdown
Member

The default server.properties file now contains the log.dirs setting and not log.dir anymore.

Co-authored-by: Mickael Maison mickael.maison@gmail.com
Co-authored-by: Katherine Farmer kfarme3@uk.ibm.com

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

The default server.properties file now contains the log.dirs setting and not log.dir anymore.

Co-authored-by: Mickael Maison <mickael.maison@gmail.com>
Co-authored-by: Katherine Farmer <kfarme3@uk.ibm.com>
@omkreddy
Copy link
Copy Markdown
Contributor

+1

1 similar comment
@harshach
Copy link
Copy Markdown

+1

@harshach harshach self-requested a review July 18, 2018 05:20
@harshach harshach merged commit 28e29a4 into apache:trunk Jul 18, 2018
@rajinisivaram
Copy link
Copy Markdown
Contributor

@mimaison Thanks for the PR, also cherry-picking to 2.0.

rajinisivaram pushed a commit that referenced this pull request Jul 18, 2018
The default server.properties file now contains the log.dirs setting and not log.dir anymore.

Co-authored-by: Mickael Maison <mickael.maison@gmail.com>
Co-authored-by: Katherine Farmer <kfarme3@uk.ibm.com>
Reviewers: Manikumar Reddy O <manikumar.reddy@gmail.com>, Sriharsha Chintalapani <sriharsha@apache.org>
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.

4 participants