Github user jorgebay commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/747#discussion_r151696373
--- Diff: gremlin-test/features/sideEffect/Sack.feature ---
@@ -0,0 +1,71 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership. The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License. You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, 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.
+
+Feature: Step - sack()
+
+ Scenario: g_withSackXhelloX_V_outE_sackXassignX_byXlabelX_inV_sack
+ Given the modern graph
+ And the traversal of
+ """
+
g.withSack("hello").V().outE().sack(Operator.assign).by(T.label).inV().sack()
+ """
+ When iterated to list
+ Then the result should be unordered
+ | result |
+ | created |
+ | knows |
+ | knows |
+ | created |
+ | created |
+ | created |
+
+ Scenario: g_withSackX0X_V_outE_sackXsumX_byXweightX_inV_sack_sum
+ Given the modern graph
+ And the traversal of
+ """
+
g.withSack(0.0).V().outE().sack(Operator.sum).by("weight").inV().sack().sum()
+ """
+ When iterated to list
+ Then the result should be unordered
+ | result |
+ | d[3.5].d |
--- End diff --
The type of the result is `BigDecimal`
```bash
gremlin>
g.withSack(0.0).V().outE().sack(Operator.sum).by("weight").inV().sack().sum().next().getClass()
==>class java.math.BigDecimal
```
We should introduce another suffix for decimals.
---
