Skip to content

[ZEPPELIN-2154] Support zeppelin.server.port in dev mode (branch-0.7)#2124

Closed
soralee wants to merge 1 commit into
apache:branch-0.7from
soralee:ZEPPELIN-2154_0.7_server_port
Closed

[ZEPPELIN-2154] Support zeppelin.server.port in dev mode (branch-0.7)#2124
soralee wants to merge 1 commit into
apache:branch-0.7from
soralee:ZEPPELIN-2154_0.7_server_port

Conversation

@soralee
Copy link
Copy Markdown
Contributor

@soralee soralee commented Mar 12, 2017

What is this PR for?

If user change zeppelin.server.port variable in zeppelin-site.xml, zeppelin doesn't work in web development on branch-0.7
#2097 PR which is same issue on master branch was already merged to master branch.

What type of PR is it?

[Bug Fix | Improvement | Documentation (README.md)]

What is the Jira issue?

How should this be tested?

  1. Change zeppelin.server.port from 8080 to 8888 (or another port) in zeppelin-site.xml
  2. Run zeppelin (bin/zeppelin-daemon.sh start)
  3. Run web development mode under zeppelin-web folder such like SERVER_PORT=8888 yarn run dev
  4. Connect localhost:9000

Screenshots (if appropriate)

[Before]
image

[After]
z_0 7_server_port_cmd
z_0 7_server_port_after

Questions:

  • Does the licenses files need update? No
  • Is there breaking changes for older versions? No
  • Does thisneeds documentation? Yes, README.md`

@Leemoonsoo
Copy link
Copy Markdown
Member

CI failure on branch-0.7 has fixed by #2103.
@soralee could you rebase this PR to branch-0.7 and see if CI passes?

@soralee
Copy link
Copy Markdown
Contributor Author

soralee commented Mar 15, 2017

Sure! Thanks @Leemoonsoo for inform it

@soralee soralee force-pushed the ZEPPELIN-2154_0.7_server_port branch 2 times, most recently from b3eff9d to 67c9e7b Compare March 17, 2017 09:05
@soralee soralee force-pushed the ZEPPELIN-2154_0.7_server_port branch from 67c9e7b to 8633842 Compare March 23, 2017 01:25
@AhyoungRyu
Copy link
Copy Markdown
Contributor

LGTM

asfgit pushed a commit that referenced this pull request Apr 8, 2017
### What is this PR for?
If user change `zeppelin.server.port` variable in zeppelin-site.xml, zeppelin doesn't work in web development on `branch-0.7`
#2097 PR which is same issue on master branch was already merged to master branch.

### What type of PR is it?
[Bug Fix | Improvement | Documentation (README.md)]

### What is the Jira issue?
* [ZEPPELIN-2154](https://issues.apache.org/jira/browse/ZEPPELIN-2154)

### How should this be tested?
1. Change `zeppelin.server.port` from 8080 to 8888 (or another port) in `zeppelin-site.xml`
2. Run zeppelin (`bin/zeppelin-daemon.sh start`)
3. Run web development mode under zeppelin-web folder such like `SERVER_PORT=8888 yarn run dev`
4. Connect localhost:9000

### Screenshots (if appropriate)
**[Before]**
![image](https://cloud.githubusercontent.com/assets/8110458/23829990/b3b7f436-0742-11e7-8bbe-c14245c42003.png)

**[After]**
![z_0 7_server_port_cmd](https://cloud.githubusercontent.com/assets/8110458/23829958/f9b9be84-0741-11e7-9db8-af4e2952b989.png)
![z_0 7_server_port_after](https://cloud.githubusercontent.com/assets/8110458/23829960/fc3fe4a8-0741-11e7-8d63-2bccf07055a1.png)

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does thi`sneeds documentation? Yes, `README.md`

Author: soralee <sora0728@zepl.com>

Closes #2124 from soralee/ZEPPELIN-2154_0.7_server_port and squashes the following commits:

8633842 [soralee] [ZEPPELIN-2154] Support zeppelin.server.port in dev mode (branch-0.7)
@AhyoungRyu
Copy link
Copy Markdown
Contributor

@soralee I just merged this PR into branch-0.7 but it's not closed automatically. Could you close it in your side?

@soralee
Copy link
Copy Markdown
Contributor Author

soralee commented Apr 8, 2017

I did, Thanks @AhyoungRyu

@soralee soralee closed this Apr 8, 2017
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.

3 participants