[GitHub] incubator-griffin issue #409: Support for HDP

[SparshSinghalHM]

Github user SparshSinghalHM commented on the issue:

https://github.com/apache/incubator-griffin/pull/409
  
@bhlx3lyx7  No, not required. We can ask for the ambari username in 
application.properties. Admin is the default user. 


---

[GitHub] incubator-griffin pull request #409: Support for HDP

[SparshSinghalHM]

Github user SparshSinghalHM commented on a diff in the pull request:

https://github.com/apache/incubator-griffin/pull/409#discussion_r215148542
  
--- Diff: 
service/src/main/java/org/apache/griffin/core/job/SparkSubmitJob.java ---
@@ -113,8 +117,13 @@ private void 
updateJobInstanceState(JobExecutionContext context) throws
 private String post2Livy() {
 String result = null;
 try {
-result = restTemplate.postForObject(livyUri, livyConfMap,
-String.class);
+HttpHeaders headers = new HttpHeaders();
+headers.setContentType(MediaType.APPLICATION_JSON);
+headers.set("X-Requested-By","admin");
--- End diff --

Yes. Can do it this way. 


---

[GitHub] incubator-griffin pull request #409: Support for HDP

[SparshSinghalHM]

GitHub user SparshSinghalHM opened a pull request:

https://github.com/apache/incubator-griffin/pull/409

Support for HDP

Added parameter for CISF protection in Ambari. Can be provided in a 
separate branch. 

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/SparshSinghalHM/incubator-griffin patch-4

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-griffin/pull/409.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #409


commit d199b6f9fd27ae9ba755e5eb00491a923094c0f5
Author: Sparsh Singhal 
Date:   2018-09-03T13:10:05Z

Support for HDP

Added parameter for CISF protection in Ambari.




---

[GitHub] incubator-griffin issue #393: Update deploy-guide.md

[SparshSinghalHM]

Github user SparshSinghalHM commented on the issue:

https://github.com/apache/incubator-griffin/pull/393
  
@bhlx3lyx7 We are no more using sparkJob.properties file for livy 
configuration as it has been replaced by sparkProperites.json after #290.


---

[GitHub] incubator-griffin pull request #397: Removed bug

[SparshSinghalHM]

GitHub user SparshSinghalHM opened a pull request:

https://github.com/apache/incubator-griffin/pull/397

Removed bug

Updated property to remove space. 

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/SparshSinghalHM/incubator-griffin patch-2

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-griffin/pull/397.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #397


commit 6e4912936debd7326547274ee25ba35d2ab733d0
Author: Sparsh Singhal 
Date:   2018-08-17T13:20:16Z

Removed bug




---

[GitHub] incubator-griffin pull request #392: Update pom.xml

[SparshSinghalHM]

Github user SparshSinghalHM closed the pull request at:

https://github.com/apache/incubator-griffin/pull/392


---

[GitHub] incubator-griffin issue #392: Update pom.xml

[SparshSinghalHM]

Github user SparshSinghalHM commented on the issue:

https://github.com/apache/incubator-griffin/pull/392
  
Hi @guoyuepeng 

Can we mention this in the document? 
Also, can we update the document with the latest updates? 

Thanks for responding.

Sparsh


---

[GitHub] incubator-griffin pull request #393: Update deploy-guide.md

[SparshSinghalHM]

GitHub user SparshSinghalHM opened a pull request:

https://github.com/apache/incubator-griffin/pull/393

Update deploy-guide.md

Updated document with new configuration files.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/SparshSinghalHM/incubator-griffin patch-3

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-griffin/pull/393.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #393


commit 474bd8d202574c816a720f5b5f73bb5cbaea06b4
Author: Sparsh Singhal 
Date:   2018-08-10T11:06:03Z

Update deploy-guide.md

Updated document with new configuration files.




---

[GitHub] incubator-griffin pull request #392: Update pom.xml

[SparshSinghalHM]

GitHub user SparshSinghalHM opened a pull request:

https://github.com/apache/incubator-griffin/pull/392

Update pom.xml

Uncomment Mysql Connector dependency.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/SparshSinghalHM/incubator-griffin patch-2

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-griffin/pull/392.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #392


commit 2d241b8bd1055f056f8ced5a327365306ce2842d
Author: Sparsh Singhal 
Date:   2018-08-10T10:54:45Z

Update pom.xml

Uncomment Mysql Connector dependency.




---

[GitHub] incubator-griffin pull request #256: Griffin 0.2.0 incubating rc3

[SparshSinghalHM]

Github user SparshSinghalHM closed the pull request at:

https://github.com/apache/incubator-griffin/pull/256


---