beam-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ted Yu (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (BEAM-1568) Ineffective null check in IsmFormat#structuralValue
Date Sun, 19 Mar 2017 21:26:41 GMT

     [ https://issues.apache.org/jira/browse/BEAM-1568?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Ted Yu updated BEAM-1568:
-------------------------
    Description: 
{code}
    public Object structuralValue(IsmRecord<V> record) throws Exception {
      checkState(record.getKeyComponents().size() == keyComponentCoders.size(),
          "Expected the number of key component coders %s "
          + "to match the number of key components %s.",
          keyComponentCoders.size(), record.getKeyComponents());

      if (record != null && consistentWithEquals()) {
{code}
record is de-referenced before the null check.

  was:
{code}
    public Object structuralValue(IsmRecord<V> record) throws Exception {
      checkState(record.getKeyComponents().size() == keyComponentCoders.size(),
          "Expected the number of key component coders %s "
          + "to match the number of key components %s.",
          keyComponentCoders.size(), record.getKeyComponents());

      if (record != null && consistentWithEquals()) {
{code}

record is de-referenced before the null check.


> Ineffective null check in IsmFormat#structuralValue
> ---------------------------------------------------
>
>                 Key: BEAM-1568
>                 URL: https://issues.apache.org/jira/browse/BEAM-1568
>             Project: Beam
>          Issue Type: Bug
>          Components: runner-dataflow
>            Reporter: Ted Yu
>            Assignee: Davor Bonaci
>            Priority: Minor
>
> {code}
>     public Object structuralValue(IsmRecord<V> record) throws Exception {
>       checkState(record.getKeyComponents().size() == keyComponentCoders.size(),
>           "Expected the number of key component coders %s "
>           + "to match the number of key components %s.",
>           keyComponentCoders.size(), record.getKeyComponents());
>       if (record != null && consistentWithEquals()) {
> {code}
> record is de-referenced before the null check.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Mime
View raw message