[GitHub] [tinkerpop] jorgebay commented on issue #1209: TINKERPOP-2305 GraphBinary: Buffer API

2019-10-16 Thread GitBox
jorgebay commented on issue #1209: TINKERPOP-2305 GraphBinary: Buffer API
URL: https://github.com/apache/tinkerpop/pull/1209#issuecomment-542704884
 
 
   Good catch about `gremlin-io-test`!
   
   I've fixed it and declared parameters `final` in the public API.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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


[tinkerpop] branch TINKERPOP-2305 updated: TINKERPOP-2035 Fix gremlin-io-test and declare params as final

2019-10-16 Thread jorgebg
This is an automated email from the ASF dual-hosted git repository.

jorgebg pushed a commit to branch TINKERPOP-2305
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git


The following commit(s) were added to refs/heads/TINKERPOP-2305 by this push:
 new c2108bb  TINKERPOP-2035 Fix gremlin-io-test and declare params as final
c2108bb is described below

commit c2108bb5ba0447dccac2e5421943c2c6fdb9216c
Author: Jorge Bay Gondra 
AuthorDate: Wed Oct 16 15:36:54 2019 +0200

TINKERPOP-2035 Fix gremlin-io-test and declare params as final
---
 .../tinkerpop/gremlin/structure/io/Buffer.java | 36 ++--
 .../gremlin/structure/io/BufferFactory.java|  4 +--
 .../tinkerpop/gremlin/driver/ser/NettyBuffer.java  | 38 +++---
 .../gremlin/driver/ser/NettyBufferFactory.java | 10 +++---
 .../scripts/generate-graphbinary-resources.groovy  |  5 ++-
 5 files changed, 48 insertions(+), 45 deletions(-)

diff --git 
a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/Buffer.java
 
b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/Buffer.java
index e370ed7..c507889 100644
--- 
a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/Buffer.java
+++ 
b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/Buffer.java
@@ -43,7 +43,7 @@ public interface Buffer {
  * @throws IndexOutOfBoundsException
  * if its out of bounds.
  */
-Buffer readerIndex(int readerIndex);
+Buffer readerIndex(final int readerIndex);
 
 /**
  * Returns the writer index of this buffer.
@@ -53,7 +53,7 @@ public interface Buffer {
 /**
  * Sets the writer index of this buffer.
  */
-Buffer writerIndex(int writerIndex);
+Buffer writerIndex(final int writerIndex);
 
 /**
  * Marks the current writer index in this buffer.
@@ -125,14 +125,14 @@ public interface Buffer {
  * @param dstIndex the first index of the destination
  * @param length   the number of bytes to transfer
  */
-Buffer readBytes(byte[] destination, int dstIndex, int length);
+Buffer readBytes(final byte[] destination, final int dstIndex, final int 
length);
 
 /**
  * Transfers this buffer's data to the specified destination starting at
  * the current reader index until the destination's position
  * reaches its limit, and advances the reader index.
  */
-Buffer readBytes(ByteBuffer dst);
+Buffer readBytes(final ByteBuffer dst);
 
 /**
  * Transfers this buffer's data to the specified stream starting at the
@@ -143,60 +143,60 @@ public interface Buffer {
  * @throws IOException
  * if the specified stream threw an exception during I/O
  */
-Buffer readBytes(OutputStream out, int length) throws IOException;
+Buffer readBytes(final OutputStream out, final int length) throws 
IOException;
 
 /**
  * Sets the specified boolean at the current writer index and advances the 
index.
  */
-Buffer writeBoolean(boolean value);
+Buffer writeBoolean(final boolean value);
 
 /**
  * Sets the specified byte at the current writer index and advances the 
index.
  */
-Buffer writeByte(int value);
+Buffer writeByte(final int value);
 
 /**
  * Sets the specified 16-bit short integer at the current writer index and 
advances the index.
  */
-Buffer writeShort(int value);
+Buffer writeShort(final int value);
 
 /**
  * Sets the specified 32-bit integer at the current writer index and 
advances the index.
  */
-Buffer writeInt(int value);
+Buffer writeInt(final int value);
 
 /**
  * Sets the specified 64-bit long integer at the current writer index and 
advances the index.
  */
-Buffer writeLong(long value);
+Buffer writeLong(final long value);
 
 /**
  * Sets the specified 32-bit floating point number at the current writer 
index and advances the index.
  */
-Buffer writeFloat(float value);
+Buffer writeFloat(final float value);
 
 /**
  * Sets the specified 64-bit floating point number at the current writer 
index and advances the index.
  */
-Buffer writeDouble(double value);
+Buffer writeDouble(final double value);
 
 /**
  * Transfers the specified source array's data to this buffer starting at 
the current writer index
  * and advances the index.
  */
-Buffer writeBytes(byte[] src);
+Buffer writeBytes(final byte[] src);
 
 /**
  * Transfers the specified source byte data to this buffer starting at the 
current writer index
  * and advances the index.
  */
-Buffer writeBytes(ByteBuffer src);
+Buffer writeBytes(final ByteBuffer src);
 
 /**
  * Transfers the specified source array's data to this buffer starting at 
the current writer index
  * and advances the index.
  */
-Buffer writeBytes(byte[] src, int srcIndex, int length);
+  

[GitHub] [tinkerpop] spmallette commented on issue #1209: TINKERPOP-2305 GraphBinary: Buffer API

2019-10-16 Thread GitBox
spmallette commented on issue #1209: TINKERPOP-2305 GraphBinary: Buffer API
URL: https://github.com/apache/tinkerpop/pull/1209#issuecomment-542663731
 
 
   I think you got the fix right, but please make sure that his works:
   
   ```text
   mvn clean install -pl :gremlin-io-test -Dio
   ```
   
   Also, needs a few `final` declarations on variables to match general code 
style. Other than that, 
   
   VOTE +1
   
   We can merge after I re-open the release branches.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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] [tinkerpop] nastra commented on issue #1209: TINKERPOP-2305 GraphBinary: Buffer API

2019-10-16 Thread GitBox
nastra commented on issue #1209: TINKERPOP-2305 GraphBinary: Buffer API
URL: https://github.com/apache/tinkerpop/pull/1209#issuecomment-542636400
 
 
   VOTE +1


This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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