You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by jo...@apache.org on 2019/02/27 14:53:39 UTC

[tinkerpop] 01/01: Merge pull request #1065 from apache/TINKERPOP-2161

This is an automated email from the ASF dual-hosted git repository.

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

commit 90528cde17d2f83bb7dc9937a6b35ce0ab9f5753
Merge: e812808 0a14205
Author: Jorge Bay Gondra <jo...@gmail.com>
AuthorDate: Wed Feb 27 15:53:33 2019 +0100

    Merge pull request #1065 from apache/TINKERPOP-2161
    
    TINKERPOP-2161 - GraphBinary: use a single buffer instead of allocators

 CHANGELOG.asciidoc                                 |   3 +
 docs/src/upgrade/release-3.4.x.asciidoc            |  12 +++
 .../driver/ser/GraphBinaryMessageSerializerV1.java |  29 +++--
 .../driver/ser/binary/GraphBinaryWriter.java       |  57 +++++-----
 .../ser/binary/RequestMessageSerializer.java       |  24 ++---
 .../ser/binary/ResponseMessageSerializer.java      |  34 +++---
 .../gremlin/driver/ser/binary/TypeSerializer.java  |   4 +-
 .../driver/ser/binary/TypeSerializerRegistry.java  |   4 +-
 .../ser/binary/types/BigDecimalSerializer.java     |  10 +-
 .../ser/binary/types/BigIntegerSerializer.java     |   5 +-
 .../driver/ser/binary/types/BindingSerializer.java |  15 +--
 .../driver/ser/binary/types/BulkSetSerializer.java |  20 +---
 .../ser/binary/types/ByteBufferSerializer.java     |   5 +-
 .../ser/binary/types/ByteCodeSerializer.java       |  46 +++-----
 .../driver/ser/binary/types/CharSerializer.java    |   5 +-
 .../driver/ser/binary/types/ClassSerializer.java   |   5 +-
 .../ser/binary/types/CollectionSerializer.java     |  19 +---
 .../driver/ser/binary/types/DateSerializer.java    |   5 +-
 .../ser/binary/types/DurationSerializer.java       |   5 +-
 .../driver/ser/binary/types/EdgeSerializer.java    |  37 +++----
 .../driver/ser/binary/types/EnumSerializer.java    |   5 +-
 .../driver/ser/binary/types/GraphSerializer.java   |  95 ++++++----------
 .../ser/binary/types/InetAddressSerializer.java    |   5 +-
 .../driver/ser/binary/types/InstantSerializer.java |   5 +-
 .../driver/ser/binary/types/LambdaSerializer.java  |  20 +---
 .../driver/ser/binary/types/ListSerializer.java    |   5 +-
 .../ser/binary/types/LocalDateSerializer.java      |   5 +-
 .../ser/binary/types/LocalDateTimeSerializer.java  |  10 +-
 .../ser/binary/types/LocalTimeSerializer.java      |   5 +-
 .../ser/binary/types/MapEntrySerializer.java       |   3 +-
 .../driver/ser/binary/types/MapSerializer.java     |  22 ++--
 .../driver/ser/binary/types/MetricsSerializer.java |  28 ++---
 .../ser/binary/types/MonthDaySerializer.java       |   5 +-
 .../ser/binary/types/OffsetDateTimeSerializer.java |  10 +-
 .../ser/binary/types/OffsetTimeSerializer.java     |  10 +-
 .../driver/ser/binary/types/PSerializer.java       |  13 +--
 .../driver/ser/binary/types/PathSerializer.java    |  18 +---
 .../driver/ser/binary/types/PeriodSerializer.java  |   5 +-
 .../ser/binary/types/PropertySerializer.java       |  13 +--
 .../driver/ser/binary/types/SetSerializer.java     |   5 +-
 .../ser/binary/types/SimpleTypeSerializer.java     |  22 ++--
 .../ser/binary/types/SingleTypeSerializer.java     |  26 ++---
 .../driver/ser/binary/types/StringSerializer.java  |   6 +-
 .../types/TraversalExplanationSerializer.java      |   3 +-
 .../binary/types/TraversalMetricsSerializer.java   |  18 +---
 .../binary/types/TraversalStrategySerializer.java  |  18 +---
 .../ser/binary/types/TraverserSerializer.java      |  17 +--
 .../driver/ser/binary/types/TreeSerializer.java    |  23 +---
 .../driver/ser/binary/types/UUIDSerializer.java    |   8 +-
 .../ser/binary/types/VertexPropertySerializer.java |  30 ++----
 .../driver/ser/binary/types/VertexSerializer.java  |  23 +---
 .../ser/binary/types/YearMonthSerializer.java      |   5 +-
 .../ser/binary/types/ZoneOffsetSerializer.java     |   5 +-
 .../ser/binary/types/ZonedDateTimeSerializer.java  |  10 +-
 .../GraphBinaryReaderWriterRoundTripTest.java      |   3 +-
 .../ser/binary/TypeSerializerFailureTests.java     |   8 +-
 .../ser/binary/TypeSerializerRegistryTest.java     |   9 +-
 .../ser/binary/types/CharSerializerTest.java       |   6 +-
 .../types/sample/SamplePersonSerializer.java       |  39 ++++---
 .../driver/GraphBinaryReaderWriterBenchmark.java   | 119 +++++++++++++++++++++
 .../gremlin/driver/GraphSONMapperBenchmark.java    | 107 ++++++++++++++++++
 .../gremlin/driver/SerializationBenchmark.java     |  18 ++--
 62 files changed, 569 insertions(+), 585 deletions(-)

diff --cc CHANGELOG.asciidoc
index 614af94,ea8205f..882ce4e
--- a/CHANGELOG.asciidoc
+++ b/CHANGELOG.asciidoc
@@@ -34,8 -34,10 +34,11 @@@ This release also includes changes fro
  * Added fallback resolver to `TypeSerializerRegistry` for GraphBinary.
  * Added easier to understand exceptions for connection problems in the Gremlin.Net driver.
  * Support configuring the type registry builder for GraphBinary.
 +* Bump to Groovy 2.5.6.
  * Release working buffers in case of failure for GraphBinary.
+ * GraphBinary: Use the same `ByteBuf` instance to write during serialization. Changed signature of write methods in type serializers.
+ * Remove unused parameter in GraphBinary's `ResponseMessageSerializer`.
+ 
  
  [[release-3-4-0]]
  === TinkerPop 3.4.0 (Release Date: January 2, 2019)