[jira] [Updated] (PIG-4554) Compress pig.script before encoding

2015-09-25 Thread Sandeep Samdaria (JIRA)

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

Sandeep Samdaria updated PIG-4554:
--
Attachment: PIG-4554-3.patch

Made changes as suggested by [~rohini]

> Compress pig.script before encoding
> ---
>
> Key: PIG-4554
> URL: https://issues.apache.org/jira/browse/PIG-4554
> Project: Pig
>  Issue Type: Improvement
>Affects Versions: 0.14.0
>Reporter: Rohini Palaniswamy
>Assignee: Sandeep Samdaria
>  Labels: newbie
> Fix For: 0.16.0
>
> Attachments: PIG-4554-2.patch, PIG-4554-3.patch, PIG-4554.patch
>
>
>   Currently we truncate the pig script (maxScriptSize = 10240) and base64 
> encode it and store in config. We should remove the truncation and store the 
> full script by compressing and then doing base64 encoding. We already do that 
> for udfcontext, etc. It will save space as it will compress really well and 
> will also give the full pig script while debugging.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (PIG-4641) Print the instance of Object without using toString()

2015-09-19 Thread Sandeep Samdaria (JIRA)

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

Sandeep Samdaria updated PIG-4641:
--
Attachment: PIG-4641.patch

Attaching the patch.
Used {{toString}} to print the value.

> Print the instance of Object without using toString()
> -
>
> Key: PIG-4641
> URL: https://issues.apache.org/jira/browse/PIG-4641
> Project: Pig
>  Issue Type: Bug
>Affects Versions: 0.13.0, 0.14.0, 0.15.0
>Reporter: songwanging
>Assignee: Sandeep Samdaria
>Priority: Minor
> Attachments: PIG-4641.patch
>
>   Original Estimate: 10m
>  Remaining Estimate: 10m
>
> In  Class: pig-0.15.0\src\org\apache\pig\scripting\js\JsFunction.java
> when printing the instance of Object "value", should use "toString()" to 
> convert it to a String instance
> private void debugConvertPigToJS(int depth, String pigType, Object value, 
> Schema schema) {
> if (LOG.isDebugEnabled()) {
> LOG.debug(indent(depth)+"converting from Pig " + pigType + " " + 
> value + " using " + stringify(schema));
> }
> }
> In the same Class, all others use toString(value) to print the instance of 
> Object "value".



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (PIG-4641) Print the instance of Object without using toString()

2015-09-19 Thread Sandeep Samdaria (JIRA)

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

Sandeep Samdaria updated PIG-4641:
--
Assignee: Sandeep Samdaria
  Status: Patch Available  (was: Open)

> Print the instance of Object without using toString()
> -
>
> Key: PIG-4641
> URL: https://issues.apache.org/jira/browse/PIG-4641
> Project: Pig
>  Issue Type: Bug
>Affects Versions: 0.15.0, 0.14.0, 0.13.0
>Reporter: songwanging
>Assignee: Sandeep Samdaria
>Priority: Minor
>   Original Estimate: 10m
>  Remaining Estimate: 10m
>
> In  Class: pig-0.15.0\src\org\apache\pig\scripting\js\JsFunction.java
> when printing the instance of Object "value", should use "toString()" to 
> convert it to a String instance
> private void debugConvertPigToJS(int depth, String pigType, Object value, 
> Schema schema) {
> if (LOG.isDebugEnabled()) {
> LOG.debug(indent(depth)+"converting from Pig " + pigType + " " + 
> value + " using " + stringify(schema));
> }
> }
> In the same Class, all others use toString(value) to print the instance of 
> Object "value".



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (PIG-4554) Compress pig.script before encoding

2015-09-18 Thread Sandeep Samdaria (JIRA)

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

Sandeep Samdaria updated PIG-4554:
--
Attachment: PIG-4554-2.patch

Made changes as suggested by Rohini.

> Compress pig.script before encoding
> ---
>
> Key: PIG-4554
> URL: https://issues.apache.org/jira/browse/PIG-4554
> Project: Pig
>  Issue Type: Improvement
>Affects Versions: 0.14.0
>Reporter: Rohini Palaniswamy
>Assignee: Sandeep Samdaria
>  Labels: newbie
> Fix For: 0.16.0
>
> Attachments: PIG-4554-2.patch, PIG-4554.patch
>
>
>   Currently we truncate the pig script (maxScriptSize = 10240) and base64 
> encode it and store in config. We should remove the truncation and store the 
> full script by compressing and then doing base64 encoding. We already do that 
> for udfcontext, etc. It will save space as it will compress really well and 
> will also give the full pig script while debugging.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (PIG-4554) Compress pig.script before encoding

2015-09-18 Thread Sandeep Samdaria (JIRA)

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

Sandeep Samdaria updated PIG-4554:
--
Status: Patch Available  (was: Open)

> Compress pig.script before encoding
> ---
>
> Key: PIG-4554
> URL: https://issues.apache.org/jira/browse/PIG-4554
> Project: Pig
>  Issue Type: Improvement
>Affects Versions: 0.14.0
>Reporter: Rohini Palaniswamy
>Assignee: Sandeep Samdaria
>  Labels: newbie
> Fix For: 0.16.0
>
> Attachments: PIG-4554.patch
>
>
>   Currently we truncate the pig script (maxScriptSize = 10240) and base64 
> encode it and store in config. We should remove the truncation and store the 
> full script by compressing and then doing base64 encoding. We already do that 
> for udfcontext, etc. It will save space as it will compress really well and 
> will also give the full pig script while debugging.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (PIG-4554) Compress pig.script before encoding

2015-09-18 Thread Sandeep Samdaria (JIRA)

[ 
https://issues.apache.org/jira/browse/PIG-4554?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14876305#comment-14876305
 ] 

Sandeep Samdaria commented on PIG-4554:
---

I have added a new variable called {{truncatedScript}} and storing the 
truncated script in that variable. 
I have one question. How can I test and verify that the Tez DAG UI is showing 
the truncated script?
I will upload the patch, once I am able to verify the above.

> Compress pig.script before encoding
> ---
>
> Key: PIG-4554
> URL: https://issues.apache.org/jira/browse/PIG-4554
> Project: Pig
>  Issue Type: Improvement
>Affects Versions: 0.14.0
>Reporter: Rohini Palaniswamy
>  Labels: newbie
> Fix For: 0.16.0
>
> Attachments: PIG-4554.patch
>
>
>   Currently we truncate the pig script (maxScriptSize = 10240) and base64 
> encode it and store in config. We should remove the truncation and store the 
> full script by compressing and then doing base64 encoding. We already do that 
> for udfcontext, etc. It will save space as it will compress really well and 
> will also give the full pig script while debugging.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (PIG-4554) Compress pig.script before encoding

2015-09-17 Thread Sandeep Samdaria (JIRA)

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

Sandeep Samdaria updated PIG-4554:
--
Status: Open  (was: Patch Available)

Misunderstood "Submit Patch" as the workflow to submit the patch. Reverting it 
back to open.

> Compress pig.script before encoding
> ---
>
> Key: PIG-4554
> URL: https://issues.apache.org/jira/browse/PIG-4554
> Project: Pig
>  Issue Type: Improvement
>Affects Versions: 0.14.0
>Reporter: Rohini Palaniswamy
>  Labels: newbie
> Fix For: 0.16.0
>
> Attachments: PIG-4554.patch
>
>
>   Currently we truncate the pig script (maxScriptSize = 10240) and base64 
> encode it and store in config. We should remove the truncation and store the 
> full script by compressing and then doing base64 encoding. We already do that 
> for udfcontext, etc. It will save space as it will compress really well and 
> will also give the full pig script while debugging.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (PIG-4554) Compress pig.script before encoding

2015-09-17 Thread Sandeep Samdaria (JIRA)

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

Sandeep Samdaria updated PIG-4554:
--
Attachment: PIG-4554.patch

Attaching the patch.

> Compress pig.script before encoding
> ---
>
> Key: PIG-4554
> URL: https://issues.apache.org/jira/browse/PIG-4554
> Project: Pig
>  Issue Type: Improvement
>Affects Versions: 0.14.0
>Reporter: Rohini Palaniswamy
>  Labels: newbie
> Fix For: 0.16.0
>
> Attachments: PIG-4554.patch
>
>
>   Currently we truncate the pig script (maxScriptSize = 10240) and base64 
> encode it and store in config. We should remove the truncation and store the 
> full script by compressing and then doing base64 encoding. We already do that 
> for udfcontext, etc. It will save space as it will compress really well and 
> will also give the full pig script while debugging.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (PIG-4554) Compress pig.script before encoding

2015-09-17 Thread Sandeep Samdaria (JIRA)

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

Sandeep Samdaria updated PIG-4554:
--
Status: Patch Available  (was: Open)

> Compress pig.script before encoding
> ---
>
> Key: PIG-4554
> URL: https://issues.apache.org/jira/browse/PIG-4554
> Project: Pig
>  Issue Type: Improvement
>Affects Versions: 0.14.0
>Reporter: Rohini Palaniswamy
>  Labels: newbie
> Fix For: 0.16.0
>
> Attachments: PIG-4554.patch
>
>
>   Currently we truncate the pig script (maxScriptSize = 10240) and base64 
> encode it and store in config. We should remove the truncation and store the 
> full script by compressing and then doing base64 encoding. We already do that 
> for udfcontext, etc. It will save space as it will compress really well and 
> will also give the full pig script while debugging.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)