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

2018-09-04 Thread 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

2018-09-04 Thread 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

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

2018-09-03 Thread 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

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

2018-08-19 Thread 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

2018-08-17 Thread 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

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

2018-08-10 Thread 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

2018-08-10 Thread 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

2018-08-10 Thread 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

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

2018-08-10 Thread 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

[GitHub] incubator-griffin pull request #257: Added support for Elastic Search 6

2018-04-13 Thread SparshSinghalHM
GitHub user SparshSinghalHM opened a pull request: https://github.com/apache/incubator-griffin/pull/257 Added support for Elastic Search 6 It is necessary to mention the content type in elastic search 6. I was not able to get metrics unless. You can merge this pull request into a

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

2018-04-13 Thread SparshSinghalHM
Github user SparshSinghalHM closed the pull request at: https://github.com/apache/incubator-griffin/pull/256 ---

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

2018-04-13 Thread SparshSinghalHM
GitHub user SparshSinghalHM opened a pull request: https://github.com/apache/incubator-griffin/pull/256 Griffin 0.2.0 incubating rc3 Adding support for Elastic Search 6. You can merge this pull request into a Git repository by running: $ git pull https://github.com/apache