Skip to content

Conversation

@silpidhua
Copy link
Member

feat: Added Spark history server apis

PR summary

Added Spark History server start, stop and get api in Node sdk

Fixes: <! -- link to issue -->

PR Checklist

Please make sure that your PR fulfills the following requirements:

  • The commit message follows the Angular Commit Message Guidelines.
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

PR Type

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • New tests
  • Build/CI related changes
  • Documentation content changes
  • Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

feat: Added Spark history server apis
@silpidhua silpidhua requested a review from ShyamalaGowri March 3, 2022 06:01
@silpidhua
Copy link
Member Author

silpidhua commented Mar 3, 2022

image

image

image

image

image

Spark History Server:
image
image
image

fix linting error
Modified HOW_TO_TEST.md
@codecov
Copy link

codecov bot commented Mar 3, 2022

Codecov Report

Merging #35 (685f2fd) into master (4f4809c) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 685f2fd differs from pull request most recent head c2ad46c. Consider uploading reports for the commit c2ad46c to get more accurate results

@@            Coverage Diff            @@
##            master       #35   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines          268       361   +93     
  Branches        36        44    +8     
=========================================
+ Hits           268       361   +93     
Impacted Files Coverage Δ
ibm-analytics-engine-api/v3.ts 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4f4809c...c2ad46c. Read the comment docs.

@silpidhua
Copy link
Member Author

image

Modification as per review comment
@silpidhua
Copy link
Member Author

* Renamed createInstanceHome to setInstanceHome

Renamed createInstanceHome to setInstanceHome

* Fixed lint error

Fixed lint error

Co-authored-by: Silpi Dhua <[email protected]>
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