One other idea I just got, how about adding a private property like this:

```c#
private bool IsSessionEnabled => !string.IsNullOrEmpty(_sessionId);
```

then this if clause could be changed to:

```c#
if (IsSessionEnabled)
```

which is easer to understand in my opinion.
But this if course only a suggestion. Ignore it if you disagree.

[ Full content available at: https://github.com/apache/tinkerpop/pull/1257 ]
This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org

Reply via email to