[GitHub] tinkerpop issue #725: TINKERPOP-1661 Docker-built documentation does not alw...

2017-10-03 Thread twilmes
Github user twilmes commented on the issue:

https://github.com/apache/tinkerpop/pull/725
  
I cleared out my images and rebuilt and all was good.

VOTE: +1


---


[GitHub] tinkerpop issue #725: TINKERPOP-1661 Docker-built documentation does not alw...

2017-10-02 Thread robertdale
Github user robertdale commented on the issue:

https://github.com/apache/tinkerpop/pull/725
  
That's fine. Now I know what to look for ;-)


---


[GitHub] tinkerpop issue #725: TINKERPOP-1661 Docker-built documentation does not alw...

2017-10-02 Thread dkuppitz
Github user dkuppitz commented on the issue:

https://github.com/apache/tinkerpop/pull/725
  
I know, Stephen and I talked briefly about it and we came to the conclusion 
that the Docker stuff is really just a developer / contributor thing and we 
shouldn't care too much about "breaking" changes within minor releases. Perhaps 
we should just send a short mail to dev@ if we make changes in the Docker image 
setups.


---


[GitHub] tinkerpop issue #725: TINKERPOP-1661 Docker-built documentation does not alw...

2017-10-02 Thread robertdale
Github user robertdale commented on the issue:

https://github.com/apache/tinkerpop/pull/725
  
@dkuppitz It was change 37a2471932229ae6a4c1f5e5f3f2d3d25bbf7fd4 that 
introduced dotnet that required the need to rebuild the images. 


---


[GitHub] tinkerpop issue #725: TINKERPOP-1661 Docker-built documentation does not alw...

2017-10-02 Thread dkuppitz
Github user dkuppitz commented on the issue:

https://github.com/apache/tinkerpop/pull/725
  
@robertdale FYI: I just fixed the download in `tp32/` and `master/` and 
rebased this PR. Rebuilding the images should no longer be an issue.


---


[GitHub] tinkerpop issue #725: TINKERPOP-1661 Docker-built documentation does not alw...

2017-10-02 Thread robertdale
Github user robertdale commented on the issue:

https://github.com/apache/tinkerpop/pull/725
  
This makes reviewing so much easier!
VOTE +1

If anyone is interested, these are the times (in seconds) for building each 
doc:
15 /usr/src/tinkermem/docs/src/recipes/duplicate-vertex.asciidoc
16 /usr/src/tinkermem/docs/src/recipes/connected-components.asciidoc
17 /usr/src/tinkermem/docs/src/recipes/shortest-path.asciidoc
18 /usr/src/tinkermem/docs/src/recipes/if-then-based-grouping.asciidoc
18 /usr/src/tinkermem/docs/src/recipes/style-guide.asciidoc
20 /usr/src/tinkermem/docs/src/recipes/between-vertices.asciidoc
20 /usr/src/tinkermem/docs/src/recipes/pagination.asciidoc
21 /usr/src/tinkermem/docs/src/recipes/traversal-component-reuse.asciidoc
22 /usr/src/tinkermem/docs/src/recipes/cycle-detection.asciidoc
23 /usr/src/tinkermem/docs/src/recipes/duplicate-edge.asciidoc
26 
/usr/src/tinkermem/docs/src/reference/implementations-tinkergraph.asciidoc
28 /usr/src/tinkermem/docs/src/recipes/centrality.asciidoc
29 
/usr/src/tinkermem/docs/src/reference/implementations-hadoop-start.asciidoc
31 /usr/src/tinkermem/docs/src/recipes/tree.asciidoc
35 /usr/src/tinkermem/docs/src/recipes/recommendation.asciidoc
35 /usr/src/tinkermem/docs/src/recipes/traversal-induced-values.asciidoc
41 /usr/src/tinkermem/docs/src/recipes/appendix.asciidoc
42 /usr/src/tinkermem/docs/src/reference/implementations-spark.asciidoc
43 /usr/src/tinkermem/docs/src/reference/gremlin-variants.asciidoc
47 /usr/src/tinkermem/docs/src/reference/implementations-hadoop-end.asciidoc
53 /usr/src/tinkermem/docs/src/reference/intro.asciidoc
56 /usr/src/tinkermem/docs/src/reference/implementations-neo4j.asciidoc
83 /usr/src/tinkermem/docs/src/reference/the-graphcomputer.asciidoc
94 /usr/src/tinkermem/docs/src/reference/the-graph.asciidoc
96 /usr/src/tinkermem/docs/src/tutorials/getting-started/index.asciidoc
98 /usr/src/tinkermem/docs/src/tutorials/the-gremlin-console/index.asciidoc
139 /usr/src/tinkermem/docs/src/dev/provider/index.asciidoc
174 /usr/src/tinkermem/docs/src/reference/implementations-giraph.asciidoc
326 /usr/src/tinkermem/docs/src/reference/gremlin-applications.asciidoc
578 /usr/src/tinkermem/docs/src/reference/the-traversal.asciidoc



---


[GitHub] tinkerpop issue #725: TINKERPOP-1661 Docker-built documentation does not alw...

2017-10-02 Thread robertdale
Github user robertdale commented on the issue:

https://github.com/apache/tinkerpop/pull/725
  
Ignore that.  Somehow my branch was reset to master.  Rebuilding...


---


[GitHub] tinkerpop issue #725: TINKERPOP-1661 Docker-built documentation does not alw...

2017-10-02 Thread robertdale
Github user robertdale commented on the issue:

https://github.com/apache/tinkerpop/pull/725
  
Hmm.. even Reference Doc isn't local.  Thought it was the first time I 
looked at it.


---


[GitHub] tinkerpop issue #725: TINKERPOP-1661 Docker-built documentation does not alw...

2017-10-02 Thread robertdale
Github user robertdale commented on the issue:

https://github.com/apache/tinkerpop/pull/725
  
I ended up using a local copy of hadoop 2.7.2 to complete my docker rebuild.

Anyway, 'Reference Documentation' is now a local link but everything else 
still points to 'tinkerpop.apache.org'



---


[GitHub] tinkerpop issue #725: TINKERPOP-1661 Docker-built documentation does not alw...

2017-10-01 Thread robertdale
Github user robertdale commented on the issue:

https://github.com/apache/tinkerpop/pull/725
  
[Started a 
thread](https://lists.apache.org/thread.html/4c8f832ce0640230614989f4d60bfd0078aa69a0e11cfd197015fde1@%3Cdev.tinkerpop.apache.org%3E)
 to try to solve my issues.


---


[GitHub] tinkerpop issue #725: TINKERPOP-1661 Docker-built documentation does not alw...

2017-10-01 Thread robertdale
Github user robertdale commented on the issue:

https://github.com/apache/tinkerpop/pull/725
  
Docker images are building this morning. Yay.


---