[jira] [Updated] (SDAP-65) ningesterpy output not printing in docker logs

2018-04-26 Thread Frank Greguska (JIRA)

 [ 
https://issues.apache.org/jira/browse/SDAP-65?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Frank Greguska updated SDAP-65:
---
Resolution: Fixed
Status: Done  (was: In Progress)

> ningesterpy output not printing in docker logs
> --
>
> Key: SDAP-65
> URL: https://issues.apache.org/jira/browse/SDAP-65
> Project: Apache Science Data Analytics Platform
>  Issue Type: Bug
>  Components: nexus
>Reporter: Frank Greguska
>Assignee: Frank Greguska
>Priority: Major
>
> The docker logs are supposed to print the stdout from the python process into 
> the logs with prefix `[ningesterpy]` but this is not currently working.
> In the current ningester docker image ningesterpy is run without the `-u` 
> flag meaning that stdout can be buffered. This can cause the `[ningesterpy]` 
> messages to not show up in the logs.
> Need to modify the entrypoint script to add `-u` to the python command.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SDAP-65) ningesterpy output not printing in docker logs

2018-04-26 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/SDAP-65?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16454908#comment-16454908
 ] 

ASF GitHub Bot commented on SDAP-65:


fgreg closed pull request #8: SDAP-65 ningesterpy output not printing in docker 
logs
URL: https://github.com/apache/incubator-sdap-ningester/pull/8
 
 
   

This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:

As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/docker/entrypoint.sh b/docker/entrypoint.sh
index 250efc0..ae80a4c 100755
--- a/docker/entrypoint.sh
+++ b/docker/entrypoint.sh
@@ -13,12 +13,19 @@
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and
 # limitations under the License.
-set -e
+set -eb
 
 NINGESTER_JAR=`find ningester/build/libs -name ningester*.jar`
 CONFIG_FILES=`find /config -name "*.yml" | awk -vORS=, '{ print $1 }'`
 GRANULE=`find /data -type f -print -quit`
 
-python -m sdap.ningesterpy 2>&1 | sed "s/^/[ningesterpy] /" &
+echo "Launching ningesterpy. Logs from this process will be prefixed with 
[ningesterpy]"
+python -u -m sdap.ningesterpy 2>&1 | stdbuf -o0 sed -e 's/^/[ningesterpy] /' &
+
+until $(curl --output /dev/null --silent --head --fail 
http://127.0.0.1:5000/healthcheck); do
+sleep 1
+done
+
+echo "Launching ningester. Logs from this process will be prefixed with 
[ningester]"
+java -Dspring.profiles.active=$1 
-Dspring.config.location=classpath:/application.yml,${CONFIG_FILES} -jar 
${NINGESTER_JAR} granule=file://${GRANULE} ${@:2} 2>&1 | sed -e 
's/^/[ningester] /'
 
-java -Dspring.profiles.active=$1 
-Dspring.config.location=classpath:/application.yml,${CONFIG_FILES} -jar 
${NINGESTER_JAR} granule=file://${GRANULE} ${@:2} | sed "s/^/[ningester] /"
\ No newline at end of file


 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> ningesterpy output not printing in docker logs
> --
>
> Key: SDAP-65
> URL: https://issues.apache.org/jira/browse/SDAP-65
> Project: Apache Science Data Analytics Platform
>  Issue Type: Bug
>  Components: nexus
>Reporter: Frank Greguska
>Assignee: Frank Greguska
>Priority: Major
>
> The docker logs are supposed to print the stdout from the python process into 
> the logs with prefix `[ningesterpy]` but this is not currently working.
> In the current ningester docker image ningesterpy is run without the `-u` 
> flag meaning that stdout can be buffered. This can cause the `[ningesterpy]` 
> messages to not show up in the logs.
> Need to modify the entrypoint script to add `-u` to the python command.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] fgreg closed pull request #8: SDAP-65 ningesterpy output not printing in docker logs

2018-04-26 Thread GitBox
fgreg closed pull request #8: SDAP-65 ningesterpy output not printing in docker 
logs
URL: https://github.com/apache/incubator-sdap-ningester/pull/8
 
 
   

This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:

As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/docker/entrypoint.sh b/docker/entrypoint.sh
index 250efc0..ae80a4c 100755
--- a/docker/entrypoint.sh
+++ b/docker/entrypoint.sh
@@ -13,12 +13,19 @@
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and
 # limitations under the License.
-set -e
+set -eb
 
 NINGESTER_JAR=`find ningester/build/libs -name ningester*.jar`
 CONFIG_FILES=`find /config -name "*.yml" | awk -vORS=, '{ print $1 }'`
 GRANULE=`find /data -type f -print -quit`
 
-python -m sdap.ningesterpy 2>&1 | sed "s/^/[ningesterpy] /" &
+echo "Launching ningesterpy. Logs from this process will be prefixed with 
[ningesterpy]"
+python -u -m sdap.ningesterpy 2>&1 | stdbuf -o0 sed -e 's/^/[ningesterpy] /' &
+
+until $(curl --output /dev/null --silent --head --fail 
http://127.0.0.1:5000/healthcheck); do
+sleep 1
+done
+
+echo "Launching ningester. Logs from this process will be prefixed with 
[ningester]"
+java -Dspring.profiles.active=$1 
-Dspring.config.location=classpath:/application.yml,${CONFIG_FILES} -jar 
${NINGESTER_JAR} granule=file://${GRANULE} ${@:2} 2>&1 | sed -e 
's/^/[ningester] /'
 
-java -Dspring.profiles.active=$1 
-Dspring.config.location=classpath:/application.yml,${CONFIG_FILES} -jar 
${NINGESTER_JAR} granule=file://${GRANULE} ${@:2} | sed "s/^/[ningester] /"
\ No newline at end of file


 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Commented] (SDAP-65) ningesterpy output not printing in docker logs

2018-04-26 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/SDAP-65?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16454886#comment-16454886
 ] 

ASF GitHub Bot commented on SDAP-65:


fgreg closed pull request #8: SDAP-65 ningesterpy output not printing in docker 
logs
URL: https://github.com/apache/incubator-sdap-ningesterpy/pull/8
 
 
   

This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:

As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/sdap/ningesterpy.py b/sdap/ningesterpy.py
index ff143ed..e036952 100644
--- a/sdap/ningesterpy.py
+++ b/sdap/ningesterpy.py
@@ -13,6 +13,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 import logging
+import sys
 import uuid
 
 from flask import Flask, request, jsonify, Response
@@ -25,8 +26,9 @@
 from werkzeug.exceptions import default_exceptions
 
 from sdap.processors.processorchain import ProcessorChain, ProcessorNotFound, 
MissingProcessorArguments
+
 logging.basicConfig(format="%(asctime)s  %(levelname)s %(process)d --- 
[%(name)s.%(funcName)s:%(lineno)d] %(message)s",
-datefmt="%Y-%m-%d %H:%M:%S")
+datefmt="%Y-%m-%d %H:%M:%S", stream=sys.stdout)
 
 applog = logging.getLogger(__name__)
 applog.setLevel(logging.INFO)
@@ -81,6 +83,11 @@ def run_processor_chain():
 return Response(result, mimetype='application/octet-stream')
 
 
+@app.route('/healthcheck', methods=['GET'], )
+def health_check():
+return ''
+
+
 def handle_error(e):
 error_id = uuid.uuid4()
 


 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> ningesterpy output not printing in docker logs
> --
>
> Key: SDAP-65
> URL: https://issues.apache.org/jira/browse/SDAP-65
> Project: Apache Science Data Analytics Platform
>  Issue Type: Bug
>  Components: nexus
>Reporter: Frank Greguska
>Assignee: Frank Greguska
>Priority: Major
>
> The docker logs are supposed to print the stdout from the python process into 
> the logs with prefix `[ningesterpy]` but this is not currently working.
> In the current ningester docker image ningesterpy is run without the `-u` 
> flag meaning that stdout can be buffered. This can cause the `[ningesterpy]` 
> messages to not show up in the logs.
> Need to modify the entrypoint script to add `-u` to the python command.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SDAP-65) ningesterpy output not printing in docker logs

2018-04-26 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/SDAP-65?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16454883#comment-16454883
 ] 

ASF GitHub Bot commented on SDAP-65:


fgreg opened a new pull request #8: SDAP-65 ningesterpy output not printing in 
docker logs
URL: https://github.com/apache/incubator-sdap-ningesterpy/pull/8
 
 
   Fixed weird bug with stdout buffering causing no logs being printed.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> ningesterpy output not printing in docker logs
> --
>
> Key: SDAP-65
> URL: https://issues.apache.org/jira/browse/SDAP-65
> Project: Apache Science Data Analytics Platform
>  Issue Type: Bug
>  Components: nexus
>Reporter: Frank Greguska
>Assignee: Frank Greguska
>Priority: Major
>
> The docker logs are supposed to print the stdout from the python process into 
> the logs with prefix `[ningesterpy]` but this is not currently working.
> In the current ningester docker image ningesterpy is run without the `-u` 
> flag meaning that stdout can be buffered. This can cause the `[ningesterpy]` 
> messages to not show up in the logs.
> Need to modify the entrypoint script to add `-u` to the python command.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] fgreg closed pull request #8: SDAP-65 ningesterpy output not printing in docker logs

2018-04-26 Thread GitBox
fgreg closed pull request #8: SDAP-65 ningesterpy output not printing in docker 
logs
URL: https://github.com/apache/incubator-sdap-ningesterpy/pull/8
 
 
   

This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:

As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/sdap/ningesterpy.py b/sdap/ningesterpy.py
index ff143ed..e036952 100644
--- a/sdap/ningesterpy.py
+++ b/sdap/ningesterpy.py
@@ -13,6 +13,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 import logging
+import sys
 import uuid
 
 from flask import Flask, request, jsonify, Response
@@ -25,8 +26,9 @@
 from werkzeug.exceptions import default_exceptions
 
 from sdap.processors.processorchain import ProcessorChain, ProcessorNotFound, 
MissingProcessorArguments
+
 logging.basicConfig(format="%(asctime)s  %(levelname)s %(process)d --- 
[%(name)s.%(funcName)s:%(lineno)d] %(message)s",
-datefmt="%Y-%m-%d %H:%M:%S")
+datefmt="%Y-%m-%d %H:%M:%S", stream=sys.stdout)
 
 applog = logging.getLogger(__name__)
 applog.setLevel(logging.INFO)
@@ -81,6 +83,11 @@ def run_processor_chain():
 return Response(result, mimetype='application/octet-stream')
 
 
+@app.route('/healthcheck', methods=['GET'], )
+def health_check():
+return ''
+
+
 def handle_error(e):
 error_id = uuid.uuid4()
 


 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] fgreg opened a new pull request #8: SDAP-65 ningesterpy output not printing in docker logs

2018-04-26 Thread GitBox
fgreg opened a new pull request #8: SDAP-65 ningesterpy output not printing in 
docker logs
URL: https://github.com/apache/incubator-sdap-ningesterpy/pull/8
 
 
   Fixed weird bug with stdout buffering causing no logs being printed.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Commented] (SDAP-65) ningesterpy output not printing in docker logs

2018-04-26 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/SDAP-65?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16454879#comment-16454879
 ] 

ASF GitHub Bot commented on SDAP-65:


fgreg closed pull request #7: SDAP-65 ningesterpy output not printing in docker 
logs
URL: https://github.com/apache/incubator-sdap-ningesterpy/pull/7
 
 
   

This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:

As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/jenkins/Jenkinsfile b/jenkins/Jenkinsfile
new file mode 100644
index 000..09cc76f
--- /dev/null
+++ b/jenkins/Jenkinsfile
@@ -0,0 +1,48 @@
+#!groovy
+
+
+pipeline{
+agent any
+
+parameters {
+string(defaultValue: '', description: 'Branch of the 
incubator-sdap-nexusproto project to build against. Leave blank to use version 
from pip specified in requirements (this does not currently work).', name: 
'buildNexusprotoBranch')
+}
+
+stages{
+stage("Build nexusproto From Source"){
+agent {
+docker {
+image 'maven:3-jdk-8'
+args '-v $HOME/.m2:/root/.m2'
+}
+}
+when{
+expression { return params.buildNexusprotoBranch ==~ 
/SDAP-\d+/ }
+}
+steps{
+git branch: "${params.buildNexusprotoBranch}", credentialsId: 
'fgreg-github', url: 'https://github.com/apache/incubator-sdap-nexusproto'
+sh './gradlew clean build tarPython'
+stash includes: 'distribution/*.tar.gz', name: 'nexusproto'
+}
+}
+stage("Build"){
+agent {
+docker {
+image 'continuumio/miniconda3:latest'
+args '-v $HOME/.conda:/.conda/pkgs'
+}
+}
+steps{
+unstash 'nexusproto'
+sh "pip install distribution/*.tar.gz"
+git branch: "${env.BRANCH_NAME}", credentialsId: 
'fgreg-github', url: 'https://github.com/apache/incubator-sdap-ningesterpy'
+sh 'python setup.py sdist'
+}
+}
+}
+post {
+always {
+archiveArtifacts artifacts: 'dist/*.tar.gz', fingerprint: true, 
onlyIfSuccessful: true
+}
+}
+}
\ No newline at end of file
diff --git a/sdap/ningesterpy.py b/sdap/ningesterpy.py
index ff143ed..e036952 100644
--- a/sdap/ningesterpy.py
+++ b/sdap/ningesterpy.py
@@ -13,6 +13,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 import logging
+import sys
 import uuid
 
 from flask import Flask, request, jsonify, Response
@@ -25,8 +26,9 @@
 from werkzeug.exceptions import default_exceptions
 
 from sdap.processors.processorchain import ProcessorChain, ProcessorNotFound, 
MissingProcessorArguments
+
 logging.basicConfig(format="%(asctime)s  %(levelname)s %(process)d --- 
[%(name)s.%(funcName)s:%(lineno)d] %(message)s",
-datefmt="%Y-%m-%d %H:%M:%S")
+datefmt="%Y-%m-%d %H:%M:%S", stream=sys.stdout)
 
 applog = logging.getLogger(__name__)
 applog.setLevel(logging.INFO)
@@ -81,6 +83,11 @@ def run_processor_chain():
 return Response(result, mimetype='application/octet-stream')
 
 
+@app.route('/healthcheck', methods=['GET'], )
+def health_check():
+return ''
+
+
 def handle_error(e):
 error_id = uuid.uuid4()
 


 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> ningesterpy output not printing in docker logs
> --
>
> Key: SDAP-65
> URL: https://issues.apache.org/jira/browse/SDAP-65
> Project: Apache Science Data Analytics Platform
>  Issue Type: Bug
>  Components: nexus
>Reporter: Frank Greguska
>Assignee: Frank Greguska
>Priority: Major
>
> The docker logs are supposed to print the stdout from the python process into 
> the logs with prefix `[ningesterpy]` but this is not currently working.
> In the current ningester docker image ningesterpy is run without the `-u` 
> flag meaning that stdout can be buffered. This can cause the `[ningesterpy]` 
> messages to not show up in the logs.
> Need to modify the entrypoint script to add `-u` to the python command.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] fgreg closed pull request #7: SDAP-65 ningesterpy output not printing in docker logs

2018-04-26 Thread GitBox
fgreg closed pull request #7: SDAP-65 ningesterpy output not printing in docker 
logs
URL: https://github.com/apache/incubator-sdap-ningesterpy/pull/7
 
 
   

This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:

As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/jenkins/Jenkinsfile b/jenkins/Jenkinsfile
new file mode 100644
index 000..09cc76f
--- /dev/null
+++ b/jenkins/Jenkinsfile
@@ -0,0 +1,48 @@
+#!groovy
+
+
+pipeline{
+agent any
+
+parameters {
+string(defaultValue: '', description: 'Branch of the 
incubator-sdap-nexusproto project to build against. Leave blank to use version 
from pip specified in requirements (this does not currently work).', name: 
'buildNexusprotoBranch')
+}
+
+stages{
+stage("Build nexusproto From Source"){
+agent {
+docker {
+image 'maven:3-jdk-8'
+args '-v $HOME/.m2:/root/.m2'
+}
+}
+when{
+expression { return params.buildNexusprotoBranch ==~ 
/SDAP-\d+/ }
+}
+steps{
+git branch: "${params.buildNexusprotoBranch}", credentialsId: 
'fgreg-github', url: 'https://github.com/apache/incubator-sdap-nexusproto'
+sh './gradlew clean build tarPython'
+stash includes: 'distribution/*.tar.gz', name: 'nexusproto'
+}
+}
+stage("Build"){
+agent {
+docker {
+image 'continuumio/miniconda3:latest'
+args '-v $HOME/.conda:/.conda/pkgs'
+}
+}
+steps{
+unstash 'nexusproto'
+sh "pip install distribution/*.tar.gz"
+git branch: "${env.BRANCH_NAME}", credentialsId: 
'fgreg-github', url: 'https://github.com/apache/incubator-sdap-ningesterpy'
+sh 'python setup.py sdist'
+}
+}
+}
+post {
+always {
+archiveArtifacts artifacts: 'dist/*.tar.gz', fingerprint: true, 
onlyIfSuccessful: true
+}
+}
+}
\ No newline at end of file
diff --git a/sdap/ningesterpy.py b/sdap/ningesterpy.py
index ff143ed..e036952 100644
--- a/sdap/ningesterpy.py
+++ b/sdap/ningesterpy.py
@@ -13,6 +13,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 import logging
+import sys
 import uuid
 
 from flask import Flask, request, jsonify, Response
@@ -25,8 +26,9 @@
 from werkzeug.exceptions import default_exceptions
 
 from sdap.processors.processorchain import ProcessorChain, ProcessorNotFound, 
MissingProcessorArguments
+
 logging.basicConfig(format="%(asctime)s  %(levelname)s %(process)d --- 
[%(name)s.%(funcName)s:%(lineno)d] %(message)s",
-datefmt="%Y-%m-%d %H:%M:%S")
+datefmt="%Y-%m-%d %H:%M:%S", stream=sys.stdout)
 
 applog = logging.getLogger(__name__)
 applog.setLevel(logging.INFO)
@@ -81,6 +83,11 @@ def run_processor_chain():
 return Response(result, mimetype='application/octet-stream')
 
 
+@app.route('/healthcheck', methods=['GET'], )
+def health_check():
+return ''
+
+
 def handle_error(e):
 error_id = uuid.uuid4()
 


 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Commented] (SDAP-65) ningesterpy output not printing in docker logs

2018-04-26 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/SDAP-65?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16454877#comment-16454877
 ] 

ASF GitHub Bot commented on SDAP-65:


fgreg opened a new pull request #7: SDAP-65 ningesterpy output not printing in 
docker logs
URL: https://github.com/apache/incubator-sdap-ningesterpy/pull/7
 
 
   There was a weird buffering problem when running under docker. Fixed it now.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> ningesterpy output not printing in docker logs
> --
>
> Key: SDAP-65
> URL: https://issues.apache.org/jira/browse/SDAP-65
> Project: Apache Science Data Analytics Platform
>  Issue Type: Bug
>  Components: nexus
>Reporter: Frank Greguska
>Assignee: Frank Greguska
>Priority: Major
>
> The docker logs are supposed to print the stdout from the python process into 
> the logs with prefix `[ningesterpy]` but this is not currently working.
> In the current ningester docker image ningesterpy is run without the `-u` 
> flag meaning that stdout can be buffered. This can cause the `[ningesterpy]` 
> messages to not show up in the logs.
> Need to modify the entrypoint script to add `-u` to the python command.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (SDAP-36) Fix failing org.apache.sdap.mudrod.ontology.process.TestLocalOntology

2018-04-26 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/SDAP-36?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16454484#comment-16454484
 ] 

ASF GitHub Bot commented on SDAP-36:


Yongyao commented on issue #11: SDAP-36 MUDROD Storage unit test suite
URL: 
https://github.com/apache/incubator-sdap-mudrod/pull/11#issuecomment-384704662
 
 
   @lewismc Kejin has added the embedded ES server as you suggested, but he got 
stuck because the client 
(https://github.com/Yongyao/incubator-sdap-mudrod/blob/SDAP-36/core/src/test/java/org/apache/sdap/mudrod/driver/ESDriverTest.java#L53)
 he got through this approach is null. I am actually a little confused with the 
embedded server approach. Why do you need it? The sample code you provided 
seems to create a ES client, which is the same as the one we created through 
ESdriver.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Fix failing org.apache.sdap.mudrod.ontology.process.TestLocalOntology
> -
>
> Key: SDAP-36
> URL: https://issues.apache.org/jira/browse/SDAP-36
> Project: Apache Science Data Analytics Platform
>  Issue Type: Task
>  Components: mudrod
>Reporter: Lewis John McGibbney
>Assignee: Lewis John McGibbney
>Priority: Major
> Fix For: 0.1
>
>
> The remote file at 
> https://raw.githubusercontent.com/ESIPFed/sweet/master/2.4/sweetAll.owl no 
> longer exists as we changed a bunch of things in SWEET including the 
> availability of any OWL serializations.
> The equivalent file (in Turtle serialization) is located at 
> https://raw.githubusercontent.com/ESIPFed/sweet/master/src/sweetAll.ttl



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] Yongyao commented on issue #11: SDAP-36 MUDROD Storage unit test suite

2018-04-26 Thread GitBox
Yongyao commented on issue #11: SDAP-36 MUDROD Storage unit test suite
URL: 
https://github.com/apache/incubator-sdap-mudrod/pull/11#issuecomment-384704662
 
 
   @lewismc Kejin has added the embedded ES server as you suggested, but he got 
stuck because the client 
(https://github.com/Yongyao/incubator-sdap-mudrod/blob/SDAP-36/core/src/test/java/org/apache/sdap/mudrod/driver/ESDriverTest.java#L53)
 he got through this approach is null. I am actually a little confused with the 
embedded server approach. Why do you need it? The sample code you provided 
seems to create a ES client, which is the same as the one we created through 
ESdriver.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services