Skip to content

Commit

Permalink
Improve coherence between 'Serializing Pipelines' examples (#1252)
Browse files Browse the repository at this point in the history
The third example in _Serializing Pipelines_ doesn't follow the second
one in a cohesive manner, which causes `kapa.ai` to give examples that
don't fit the explanation text. See [the relevant Slack
discussion](https://hazelcast.slack.com/archives/C035HQET5/p1723034714614779).
  • Loading branch information
burakgok committed Aug 20, 2024
1 parent 839d3ea commit 8cb1b78
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions docs/modules/pipelines/pages/serialization.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ its flipside: it is easy to overlook what’s going on.

If the lambda references a variable in the outer scope, the variable is
captured and must also be serializable. If it references an instance
variable of the enclosing class, it implicitly captures this so the
variable of the enclosing class, it implicitly captures `this` so the
entire class will be serialized. For example, this will fail because
`JetJob1` does not implement `Serializable`:

Expand Down Expand Up @@ -75,8 +75,10 @@ surrounding class. We can simply achieve this by assigning to a local
variable, then referring to that variable inside the lambda:

```java
class JetJob3 {
class JetJob3 implements Serializable {
private String instanceVar;
// A non-serializable field.
private OutputStream fileOut;

Pipeline buildPipeline() {
Pipeline p = Pipeline.create();
Expand Down

0 comments on commit 8cb1b78

Please sign in to comment.