You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pig.apache.org by "Benjamin Francisoud (JIRA)" <ji...@apache.org> on 2008/02/05 16:24:08 UTC

[jira] Created: (PIG-92) PigContext NullPointerException because of uninitialize conf

PigContext NullPointerException because of uninitialize conf
------------------------------------------------------------

                 Key: PIG-92
                 URL: https://issues.apache.org/jira/browse/PIG-92
             Project: Pig
          Issue Type: Bug
          Components: impl
    Affects Versions: 0.1.0
            Reporter: Benjamin Francisoud


This simple code throw an NPE
{code:java}
final PigContext pigContext = new PigContext(ExecType.MAPREDUCE);
pigContext.getConf().putAll(properties);
{code}

Because in PigContext.java:

{code:java}
transient private Properties conf = null;
public void connect() throws ExecException {
    ... 
    conf = new Properties();
    ....
}
{code}

Simple patch:

{code:java}
transient private Properties conf = new Properties();
public void connect() throws ExecException {
    ... 
}
{code}

This is regression already fix in PIG-69.
Introduce with PIG-32


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (PIG-92) PigContext NullPointerException because of uninitialize conf

Posted by "Olga Natkovich (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/PIG-92?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12566734#action_12566734 ] 

Olga Natkovich commented on PIG-92:
-----------------------------------

ok, could you then regenerate your patch from the latest svn, thanks.

> PigContext NullPointerException because of uninitialize conf
> ------------------------------------------------------------
>
>                 Key: PIG-92
>                 URL: https://issues.apache.org/jira/browse/PIG-92
>             Project: Pig
>          Issue Type: Bug
>          Components: impl
>    Affects Versions: 0.1.0
>            Reporter: Benjamin Francisoud
>         Attachments: PIG-92-v01.patch
>
>
> This simple code throw an NPE
> {code:java}
> final PigContext pigContext = new PigContext(ExecType.MAPREDUCE);
> pigContext.getConf().putAll(properties);
> {code}
> Because in PigContext.java:
> {code:java}
> transient private Properties conf = null;
> public void connect() throws ExecException {
>     ... 
>     conf = new Properties();
>     ....
> }
> {code}
> Simple patch:
> {code:java}
> transient private Properties conf = new Properties();
> public void connect() throws ExecException {
>     ... 
> }
> {code}
> This is regression already fix in PIG-69.
> Introduce with PIG-32

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (PIG-92) PigContext NullPointerException because of uninitialize conf

Posted by "Benjamin Francisoud (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/PIG-92?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12566606#action_12566606 ] 

Benjamin Francisoud commented on PIG-92:
----------------------------------------

> Does this patch still needs to be applied?

Unless I missed something, I don't think the null pointer has been fix...



> PigContext NullPointerException because of uninitialize conf
> ------------------------------------------------------------
>
>                 Key: PIG-92
>                 URL: https://issues.apache.org/jira/browse/PIG-92
>             Project: Pig
>          Issue Type: Bug
>          Components: impl
>    Affects Versions: 0.1.0
>            Reporter: Benjamin Francisoud
>         Attachments: PIG-92-v01.patch
>
>
> This simple code throw an NPE
> {code:java}
> final PigContext pigContext = new PigContext(ExecType.MAPREDUCE);
> pigContext.getConf().putAll(properties);
> {code}
> Because in PigContext.java:
> {code:java}
> transient private Properties conf = null;
> public void connect() throws ExecException {
>     ... 
>     conf = new Properties();
>     ....
> }
> {code}
> Simple patch:
> {code:java}
> transient private Properties conf = new Properties();
> public void connect() throws ExecException {
>     ... 
> }
> {code}
> This is regression already fix in PIG-69.
> Introduce with PIG-32

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (PIG-92) PigContext NullPointerException because of uninitialize conf

Posted by "Olga Natkovich (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/PIG-92?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12566339#action_12566339 ] 

Olga Natkovich commented on PIG-92:
-----------------------------------

Does this patch still needs to be applied?

>From the comment above, it seems that it has already been adressed in another bug.

> PigContext NullPointerException because of uninitialize conf
> ------------------------------------------------------------
>
>                 Key: PIG-92
>                 URL: https://issues.apache.org/jira/browse/PIG-92
>             Project: Pig
>          Issue Type: Bug
>          Components: impl
>    Affects Versions: 0.1.0
>            Reporter: Benjamin Francisoud
>         Attachments: PIG-92-v01.patch
>
>
> This simple code throw an NPE
> {code:java}
> final PigContext pigContext = new PigContext(ExecType.MAPREDUCE);
> pigContext.getConf().putAll(properties);
> {code}
> Because in PigContext.java:
> {code:java}
> transient private Properties conf = null;
> public void connect() throws ExecException {
>     ... 
>     conf = new Properties();
>     ....
> }
> {code}
> Simple patch:
> {code:java}
> transient private Properties conf = new Properties();
> public void connect() throws ExecException {
>     ... 
> }
> {code}
> This is regression already fix in PIG-69.
> Introduce with PIG-32

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (PIG-92) PigContext NullPointerException because of uninitialize conf

Posted by "Alan Gates (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/PIG-92?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Alan Gates resolved PIG-92.
---------------------------

       Resolution: Fixed
    Fix Version/s: 0.1.0

Patch contributed by Benjamin Francisoud committed as revision 620878.  Thanks Benjamin.

> PigContext NullPointerException because of uninitialize conf
> ------------------------------------------------------------
>
>                 Key: PIG-92
>                 URL: https://issues.apache.org/jira/browse/PIG-92
>             Project: Pig
>          Issue Type: Bug
>          Components: impl
>    Affects Versions: 0.1.0
>            Reporter: Benjamin Francisoud
>             Fix For: 0.1.0
>
>         Attachments: PIG-92-v01.patch, PIG-92-v02.patch
>
>
> This simple code throw an NPE
> {code:java}
> final PigContext pigContext = new PigContext(ExecType.MAPREDUCE);
> pigContext.getConf().putAll(properties);
> {code}
> Because in PigContext.java:
> {code:java}
> transient private Properties conf = null;
> public void connect() throws ExecException {
>     ... 
>     conf = new Properties();
>     ....
> }
> {code}
> Simple patch:
> {code:java}
> transient private Properties conf = new Properties();
> public void connect() throws ExecException {
>     ... 
> }
> {code}
> This is regression already fix in PIG-69.
> Introduce with PIG-32

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (PIG-92) PigContext NullPointerException because of uninitialize conf

Posted by "Benjamin Francisoud (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/PIG-92?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Benjamin Francisoud updated PIG-92:
-----------------------------------

    Attachment: PIG-92-v01.patch

Patch as explain in the jira description.

> PigContext NullPointerException because of uninitialize conf
> ------------------------------------------------------------
>
>                 Key: PIG-92
>                 URL: https://issues.apache.org/jira/browse/PIG-92
>             Project: Pig
>          Issue Type: Bug
>          Components: impl
>    Affects Versions: 0.1.0
>            Reporter: Benjamin Francisoud
>         Attachments: PIG-92-v01.patch
>
>
> This simple code throw an NPE
> {code:java}
> final PigContext pigContext = new PigContext(ExecType.MAPREDUCE);
> pigContext.getConf().putAll(properties);
> {code}
> Because in PigContext.java:
> {code:java}
> transient private Properties conf = null;
> public void connect() throws ExecException {
>     ... 
>     conf = new Properties();
>     ....
> }
> {code}
> Simple patch:
> {code:java}
> transient private Properties conf = new Properties();
> public void connect() throws ExecException {
>     ... 
> }
> {code}
> This is regression already fix in PIG-69.
> Introduce with PIG-32

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (PIG-92) PigContext NullPointerException because of uninitialize conf

Posted by "Benjamin Francisoud (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/PIG-92?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Benjamin Francisoud updated PIG-92:
-----------------------------------

    Attachment: PIG-92-v02.patch

new patch against trunk r620508

> PigContext NullPointerException because of uninitialize conf
> ------------------------------------------------------------
>
>                 Key: PIG-92
>                 URL: https://issues.apache.org/jira/browse/PIG-92
>             Project: Pig
>          Issue Type: Bug
>          Components: impl
>    Affects Versions: 0.1.0
>            Reporter: Benjamin Francisoud
>         Attachments: PIG-92-v01.patch, PIG-92-v02.patch
>
>
> This simple code throw an NPE
> {code:java}
> final PigContext pigContext = new PigContext(ExecType.MAPREDUCE);
> pigContext.getConf().putAll(properties);
> {code}
> Because in PigContext.java:
> {code:java}
> transient private Properties conf = null;
> public void connect() throws ExecException {
>     ... 
>     conf = new Properties();
>     ....
> }
> {code}
> Simple patch:
> {code:java}
> transient private Properties conf = new Properties();
> public void connect() throws ExecException {
>     ... 
> }
> {code}
> This is regression already fix in PIG-69.
> Introduce with PIG-32

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.