[GitHub] chrismattmann commented on a change in pull request #129: Replaced Processwrapper indexing method with SolrIndexer

2018-05-18 Thread GitBox
chrismattmann commented on a change in pull request #129: Replaced Processwrapper indexing method with SolrIndexer URL: https://github.com/apache/drat/pull/129#discussion_r189425907 ## File path: proteus/src/main/java/backend/FileConstants.java ## @@ -32,7 +34,15 @@

[GitHub] chrismattmann commented on a change in pull request #129: Replaced Processwrapper indexing method with SolrIndexer

2018-05-18 Thread GitBox
chrismattmann commented on a change in pull request #129: Replaced Processwrapper indexing method with SolrIndexer URL: https://github.com/apache/drat/pull/129#discussion_r189425897 ## File path: proteus/src/main/java/backend/ProcessDratWrapper.java ## @@ -104,8 +115,20

[GitHub] ahmedifhaam commented on a change in pull request #129: Replaced Processwrapper indexing method with SolrIndexer

2018-05-18 Thread GitBox
ahmedifhaam commented on a change in pull request #129: Replaced Processwrapper indexing method with SolrIndexer URL: https://github.com/apache/drat/pull/129#discussion_r189401238 ## File path: proteus/src/main/java/backend/ProcessDratWrapper.java ## @@ -175,7 +190,12 @@

[GitHub] ahmedifhaam commented on a change in pull request #129: Replaced Processwrapper indexing method with SolrIndexer

2018-05-18 Thread GitBox
ahmedifhaam commented on a change in pull request #129: Replaced Processwrapper indexing method with SolrIndexer URL: https://github.com/apache/drat/pull/129#discussion_r189400979 ## File path: proteus/src/main/java/backend/ProcessDratWrapper.java ## @@ -104,8 +109,18 @@

[GitHub] ahmedifhaam commented on a change in pull request #129: Replaced Processwrapper indexing method with SolrIndexer

2018-05-18 Thread GitBox
ahmedifhaam commented on a change in pull request #129: Replaced Processwrapper indexing method with SolrIndexer URL: https://github.com/apache/drat/pull/129#discussion_r189400645 ## File path: proteus/src/main/java/backend/FileConstants.java ## @@ -32,7 +34,15 @@

Re: Joining on board

2018-05-18 Thread Ahmed Ifhaam
hi all, This week i'm Working on moving the main operations of drat from process wrapper to either API calls or direct java implementation. Already have completed indexing function and also have made the pull request [1]. I expect to finish this step by may 25th. It would be great if you can

[GitHub] ahmedifhaam commented on issue #129: Replaced Processwrapper indexing method with SolrIndexer

2018-05-18 Thread GitBox
ahmedifhaam commented on issue #129: Replaced Processwrapper indexing method with SolrIndexer URL: https://github.com/apache/drat/pull/129#issuecomment-390250812 Yes tested on drat repository it self.Working fine compared the results from the last master same results

[GitHub] ahmedifhaam commented on issue #129: Replaced Processwrapper indexing method with SolrIndexer

2018-05-18 Thread GitBox
ahmedifhaam commented on issue #129: Replaced Processwrapper indexing method with SolrIndexer URL: https://github.com/apache/drat/pull/129#issuecomment-390250812 Yes tested on drat repository it self working fine This is an

[GitHub] chrismattmann commented on a change in pull request #129: Replaced Processwrapper indexing method with SolrIndexer

2018-05-18 Thread GitBox
chrismattmann commented on a change in pull request #129: Replaced Processwrapper indexing method with SolrIndexer URL: https://github.com/apache/drat/pull/129#discussion_r189308433 ## File path: proteus/src/main/java/backend/ProcessDratWrapper.java ## @@ -104,8 +109,18

[GitHub] chrismattmann commented on a change in pull request #129: Replaced Processwrapper indexing method with SolrIndexer

2018-05-18 Thread GitBox
chrismattmann commented on a change in pull request #129: Replaced Processwrapper indexing method with SolrIndexer URL: https://github.com/apache/drat/pull/129#discussion_r189311225 ## File path: webapps/solr-webapp/src/main/webapp/WEB-INF/web.xml ## @@ -36,8 +36,9 @@

[GitHub] chrismattmann commented on a change in pull request #129: Replaced Processwrapper indexing method with SolrIndexer

2018-05-18 Thread GitBox
chrismattmann commented on a change in pull request #129: Replaced Processwrapper indexing method with SolrIndexer URL: https://github.com/apache/drat/pull/129#discussion_r189307034 ## File path: proteus/src/main/java/backend/FileConstants.java ## @@ -32,7 +34,15 @@

[GitHub] chrismattmann commented on a change in pull request #129: Replaced Processwrapper indexing method with SolrIndexer

2018-05-18 Thread GitBox
chrismattmann commented on a change in pull request #129: Replaced Processwrapper indexing method with SolrIndexer URL: https://github.com/apache/drat/pull/129#discussion_r189308674 ## File path: proteus/src/main/java/backend/ProcessDratWrapper.java ## @@ -104,8 +109,18