Uploaded image for project: 'Flink'
  1. Flink
  2. FLINK-6874

Static and transient fields ignored for POJOs

        Details

        • Type: Improvement
        • Status: Closed
        • Priority: Trivial
        • Resolution: Fixed
        • Affects Version/s: 1.4.0
        • Fix Version/s: 1.3.2
        • Component/s: Documentation
        • Labels:
          None

          Description

          Update dev/types_serialization.html to note that static and transient fields are ignored when validating a POJO (TypeExtractor#analyzePojo calls #getAllDeclaredFields which ignores transient and static fields).

          "All fields in the class (and all superclasses) are either public (and non-final) or have a public getter- and a setter- method that follows the Java beans naming conventions for getters and setters."

            Issue Links

            links to

            Web Link GitHub Pull Request #4091

              Activity

              Descending order - Click to sort in ascending order
              Hide
              Permalink
              tzulitai Tzu-Li (Gordon) Tai added a comment -

              Fixed for master via 20838efc5dfe0a4be917682f936f447d8df37986.
              Fixed for 1.322 via 1d0a47bdde922dbdcb28b08be3706f7c3624d485.

              Show
              tzulitai Tzu-Li (Gordon) Tai added a comment - Fixed for master via 20838efc5dfe0a4be917682f936f447d8df37986. Fixed for 1.322 via 1d0a47bdde922dbdcb28b08be3706f7c3624d485.
              Hide
              Permalink
              githubbot ASF GitHub Bot added a comment -

              Github user asfgit closed the pull request at:

              https://github.com/apache/flink/pull/4091

              Show
              githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/4091
              Hide
              Permalink
              githubbot ASF GitHub Bot added a comment -

              Github user tzulitai commented on the issue:

              https://github.com/apache/flink/pull/4091

              Thanks for the PR @greghogan.
              LGTM! Will merge this ..

              Show
              githubbot ASF GitHub Bot added a comment - Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/4091 Thanks for the PR @greghogan. LGTM! Will merge this ..
              Hide
              Permalink
              githubbot ASF GitHub Bot added a comment -

              GitHub user greghogan opened a pull request:

              https://github.com/apache/flink/pull/4091

              FLINK-6874 [docs] Static and transient fields ignored for POJOs

              Note that static and transient fields are ignored when TypeExtrator validates a POJO.

              You can merge this pull request into a Git repository by running:

              $ git pull https://github.com/greghogan/flink 6874_static_and_transient_fields_ignored_for_pojos

              Alternatively you can review and apply these changes as the patch at:

              https://github.com/apache/flink/pull/4091.patch

              To close this pull request, make a commit to your master/trunk branch
              with (at least) the following in the commit message:

              This closes #4091

              commit 64c4669c6c937a4611076f5bfcc2f334f70ca46e
              Author: Greg Hogan <code@greghogan.com>
              Date: 2017-06-08T14:51:49Z

              FLINK-6874 [docs] Static and transient fields ignored for POJOs

              Note that static and transient fields are ignored when TypeExtrator
              validates a POJO.

              Show
              githubbot ASF GitHub Bot added a comment - GitHub user greghogan opened a pull request: https://github.com/apache/flink/pull/4091 FLINK-6874 [docs] Static and transient fields ignored for POJOs Note that static and transient fields are ignored when TypeExtrator validates a POJO. You can merge this pull request into a Git repository by running: $ git pull https://github.com/greghogan/flink 6874_static_and_transient_fields_ignored_for_pojos Alternatively you can review and apply these changes as the patch at: https://github.com/apache/flink/pull/4091.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #4091 commit 64c4669c6c937a4611076f5bfcc2f334f70ca46e Author: Greg Hogan <code@greghogan.com> Date: 2017-06-08T14:51:49Z FLINK-6874 [docs] Static and transient fields ignored for POJOs Note that static and transient fields are ignored when TypeExtrator validates a POJO.

                People

                • Assignee:
                  greghogan Greg Hogan
                  Reporter:
                  greghogan Greg Hogan
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Development