Skip to content

Conversation

@jongyoul
Copy link
Member

@jongyoul jongyoul commented Jul 1, 2016

What is this PR for?

Running ZeppelinServer.main normally. It will enable user to debug ZeppelinServer easily.

What type of PR is it?

[Bug Fix]

Todos

  • - Fix configuration values

What is the Jira issue?

N/A

How should this be tested?

  • mvn clean package -DskipTests
  • Debug ZeppelinServer in your IDE

Screenshots (if appropriate)

Questions:

  • Does the licenses files need update? No
  • Is there breaking changes for older versions? No
  • Does this needs documentation? No

@jongyoul
Copy link
Member Author

jongyoul commented Jul 1, 2016

It works in Intellij. Could anyone check if it works on Eclipse?

@jongyoul jongyoul closed this Jul 1, 2016
@jongyoul jongyoul reopened this Jul 1, 2016
@jongyoul
Copy link
Member Author

jongyoul commented Jul 1, 2016

BTW, it has a problem to run mvn verify. I'll fix it

@jongyoul
Copy link
Member Author

jongyoul commented Jul 4, 2016

Ready to review.

@asfgit asfgit closed this in ebe8855 Jul 4, 2016
PhilippGrulich pushed a commit to SWC-SENSE/zeppelin that referenced this pull request Aug 8, 2016
### What is this PR for?
Running ZeppelinServer.main normally. It will enable user to debug ZeppelinServer easily.

### What type of PR is it?
[Bug Fix]

### Todos
* [x] - Fix configuration values

### What is the Jira issue?
N/A

### How should this be tested?
- mvn clean package -DskipTests
- Debug ZeppelinServer in your IDE

### Screenshots (if appropriate)

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No

Author: Jongyoul Lee <[email protected]>

Closes apache#1112 from jongyoul/minor-enable-launch-z-debugger-mode and squashes the following commits:

375a95b [Jongyoul Lee] Fixed to pass the tests
eb1f733 [Jongyoul Lee] Fixed to pass the tests
161ffce [Jongyoul Lee] Removed unused import
d24b854 [Jongyoul Lee] Set some configuration to pass `mvn verify`
921a293 [Jongyoul Lee] Changed the default value of some configurations
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.

1 participant