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

          Activity

          Hide
          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.
          Hide
          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
          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
          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.

            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