Skip to content

[SPARK-43181][SQL] Show UI WebURL in spark-sql shell#40844

Closed
panbingkun wants to merge 1 commit intoapache:masterfrom
panbingkun:SPARK-43181
Closed

[SPARK-43181][SQL] Show UI WebURL in spark-sql shell#40844
panbingkun wants to merge 1 commit intoapache:masterfrom
panbingkun:SPARK-43181

Conversation

@panbingkun
Copy link
Contributor

@panbingkun panbingkun commented Apr 19, 2023

What changes were proposed in this pull request?

The pr aims to display the Spark WEB UI address when spark-sql startup.

Why are the changes needed?

Promoting user experience.

Like spark-shell, it would be great if spark-sql show the UI information.

$ bin/spark-sql
Setting default log level to "WARN".
To adjust logging level use sc.setLogLevel(newLevel). For SparkR, use setLogLevel(newLevel).
23/05/07 13:58:26 WARN NativeCodeLoader: Unable to load native-hadoop library for your platform... using builtin-java classes where applicable
23/05/07 13:58:27 WARN HiveConf: HiveConf of name hive.stats.jdbc.timeout does not exist
23/05/07 13:58:27 WARN HiveConf: HiveConf of name hive.stats.retries.wait does not exist
23/05/07 13:58:28 WARN ObjectStore: Version information not found in metastore. hive.metastore.schema.verification is not enabled so recording the schema version 2.3.0
23/05/07 13:58:28 WARN ObjectStore: setMetaStoreSchemaVersion called but recording version is disabled: version = 2.3.0, comment = Set by MetaStore dongjoon@127.0.0.1
Spark Web UI available at http://localhost:4040
Spark master: local[*], Application Id: local-1683493106875
spark-sql (default)>

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass GA & Manually test.

@github-actions github-actions bot added the SQL label Apr 19, 2023
@LuciferYang
Copy link
Contributor

cc @HyukjinKwon FYI

@panbingkun panbingkun changed the title [SPARK-43181][SQL] spark-sql console should display the Spark WEB UI address [SPARK-43181][SQL] Spark-sql console should display the Spark WEB UI address Apr 19, 2023
Copy link
Member

@Hisoka-X Hisoka-X left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dongjoon-hyun dongjoon-hyun changed the title [SPARK-43181][SQL] Spark-sql console should display the Spark WEB UI address [SPARK-43181][SQL] Show Web UI address in spark-sql shell May 7, 2023
Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To @panbingkun , if you don't mind, please don't expose your internal fork info in Apache Spark repository. For example, the PR description will be a commit log in Apache Spark repository, so we had better remove those information from your PR description like your company's Spark 4-digit version name and company-name postfix.

FYI, for this PR, I clean them up.

@dongjoon-hyun dongjoon-hyun changed the title [SPARK-43181][SQL] Show Web UI address in spark-sql shell [SPARK-43181][SQL] Show UI WebURL in spark-sql shell May 7, 2023
@dongjoon-hyun
Copy link
Member

Merged to master for Apache Spark 3.5.0.
Thank you, @panbingkun , @LuciferYang , @Hisoka-X .

@panbingkun
Copy link
Contributor Author

To @panbingkun , if you don't mind, please don't expose your internal fork info in Apache Spark repository. For example, the PR description will be a commit log in Apache Spark repository, so we had better remove those information from your PR description like your company's Spark 4-digit version name and company-name postfix.

FYI, for this PR, I clean them up.

OK,Thanks.

LuciferYang pushed a commit to LuciferYang/spark that referenced this pull request May 10, 2023
### What changes were proposed in this pull request?
The pr aims to display `the Spark WEB UI address` when spark-sql startup.

### Why are the changes needed?
Promoting user experience.

Like `spark-shell`, it would be great if `spark-sql` show the UI information.

```
$ bin/spark-sql
Setting default log level to "WARN".
To adjust logging level use sc.setLogLevel(newLevel). For SparkR, use setLogLevel(newLevel).
23/05/07 13:58:26 WARN NativeCodeLoader: Unable to load native-hadoop library for your platform... using builtin-java classes where applicable
23/05/07 13:58:27 WARN HiveConf: HiveConf of name hive.stats.jdbc.timeout does not exist
23/05/07 13:58:27 WARN HiveConf: HiveConf of name hive.stats.retries.wait does not exist
23/05/07 13:58:28 WARN ObjectStore: Version information not found in metastore. hive.metastore.schema.verification is not enabled so recording the schema version 2.3.0
23/05/07 13:58:28 WARN ObjectStore: setMetaStoreSchemaVersion called but recording version is disabled: version = 2.3.0, comment = Set by MetaStore dongjoon127.0.0.1
Spark Web UI available at http://localhost:4040
Spark master: local[*], Application Id: local-1683493106875
spark-sql (default)>
```

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?
Pass GA & Manually test.

Closes apache#40844 from panbingkun/SPARK-43181.

Authored-by: panbingkun <pbk1982@gmail.com>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants