[GitHub] spark pull request: [SPARK-13400] Stop using deprecated Octal esca...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/spark/pull/11584


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-13400] Stop using deprecated Octal esca...

[rxin]

Github user rxin commented on the pull request:

https://github.com/apache/spark/pull/11584#issuecomment-194007682
  
Thanks - merging in master.



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-13400] Stop using deprecated Octal esca...

[dongjoon-hyun]

Github user dongjoon-hyun commented on the pull request:

https://github.com/apache/spark/pull/11584#issuecomment-194004205
  
At last, it passed and there is no `Octal` warning.
Thank you all.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-13400] Stop using deprecated Octal esca...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/11584#issuecomment-194003516
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52685/
Test PASSed.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-13400] Stop using deprecated Octal esca...

[AmplabJenkins]

Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/11584#issuecomment-194003514
  
Merged build finished. Test PASSed.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-13400] Stop using deprecated Octal esca...

[SparkQA]

Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/11584#issuecomment-194002875
  
**[Test build #52685 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52685/consoleFull)**
 for PR 11584 at commit 
[`f7fd01a`](https://github.com/apache/spark/commit/f7fd01ab4771537946d448c9d91e83050873a115).
 * This patch passes all tests.
 * This patch merges cleanly.
 * This patch adds no public classes.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-13400] Stop using deprecated Octal esca...

[dongjoon-hyun]

Github user dongjoon-hyun commented on the pull request:

https://github.com/apache/spark/pull/11584#issuecomment-193987270
  
Thank you for confirming that, @holdenk ! :)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-13400] Stop using deprecated Octal esca...

[holdenk]

Github user holdenk commented on the pull request:

https://github.com/apache/spark/pull/11584#issuecomment-193982208
  
Looking at the build logs no Octal deprecation warnings - (still waiting on 
the tests to finish but build part finished) :)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-13400] Stop using deprecated Octal esca...

[dongjoon-hyun]

Github user dongjoon-hyun commented on the pull request:

https://github.com/apache/spark/pull/11584#issuecomment-193953311
  
Yes. We should do double check Jenkins log although I've seen those two 
lines in my local machine, too.
Thank you, @rxin and @holdenk .


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-13400] Stop using deprecated Octal esca...

[holdenk]

Github user holdenk commented on the pull request:

https://github.com/apache/spark/pull/11584#issuecomment-193942199
  
LGTM pending tests, we should of course do a quick search in the resulting 
build log and make sure no octal warnings show up but these are the two I had 
seen when building locally.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-13400] Stop using deprecated Octal esca...

[rxin]

Github user rxin commented on the pull request:

https://github.com/apache/spark/pull/11584#issuecomment-193941994
  
LGTM pending tests.



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-13400] Stop using deprecated Octal esca...

[rxin]

Github user rxin commented on a diff in the pull request:

https://github.com/apache/spark/pull/11584#discussion_r55417544
  
--- Diff: 
sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveQlSuite.scala ---
@@ -71,7 +71,7 @@ class HiveQlSuite extends SparkFunSuite with 
BeforeAndAfterAll {
   CatalogColumn("dt", "string", comment = Some("date type")) ::
   CatalogColumn("hour", "string", comment = Some("hour of the day")) 
:: Nil)
 assert(desc.storage.serdeProperties ==
-  Map((serdeConstants.SERIALIZATION_FORMAT, "\054"), 
(serdeConstants.FIELD_DELIM, "\054")))
+  Map((serdeConstants.SERIALIZATION_FORMAT, "\u002C"), 
(serdeConstants.FIELD_DELIM, "\u002C")))
--- End diff --

ah ok - thanks.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-13400] Stop using deprecated Octal esca...

[holdenk]

Github user holdenk commented on a diff in the pull request:

https://github.com/apache/spark/pull/11584#discussion_r55417136
  
--- Diff: 
sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveQlSuite.scala ---
@@ -71,7 +71,7 @@ class HiveQlSuite extends SparkFunSuite with 
BeforeAndAfterAll {
   CatalogColumn("dt", "string", comment = Some("date type")) ::
   CatalogColumn("hour", "string", comment = Some("hour of the day")) 
:: Nil)
 assert(desc.storage.serdeProperties ==
-  Map((serdeConstants.SERIALIZATION_FORMAT, "\054"), 
(serdeConstants.FIELD_DELIM, "\054")))
+  Map((serdeConstants.SERIALIZATION_FORMAT, "\u002C"), 
(serdeConstants.FIELD_DELIM, "\u002C")))
--- End diff --

So "\u002C" == "\054" returns true, and 054 is in octal which is 44 in 
decimal.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-13400] Stop using deprecated Octal esca...

[SparkQA]

Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/11584#issuecomment-193940888
  
**[Test build #52685 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52685/consoleFull)**
 for PR 11584 at commit 
[`f7fd01a`](https://github.com/apache/spark/commit/f7fd01ab4771537946d448c9d91e83050873a115).


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-13400] Stop using deprecated Octal esca...

[rxin]

Github user rxin commented on a diff in the pull request:

https://github.com/apache/spark/pull/11584#discussion_r55416809
  
--- Diff: 
sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveQlSuite.scala ---
@@ -71,7 +71,7 @@ class HiveQlSuite extends SparkFunSuite with 
BeforeAndAfterAll {
   CatalogColumn("dt", "string", comment = Some("date type")) ::
   CatalogColumn("hour", "string", comment = Some("hour of the day")) 
:: Nil)
 assert(desc.storage.serdeProperties ==
-  Map((serdeConstants.SERIALIZATION_FORMAT, "\054"), 
(serdeConstants.FIELD_DELIM, "\054")))
+  Map((serdeConstants.SERIALIZATION_FORMAT, "\u002C"), 
(serdeConstants.FIELD_DELIM, "\u002C")))
--- End diff --

maybe i'm not understanding this correctly, but isn't 2C = 44?



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request: [SPARK-13400] Stop using deprecated Octal esca...

[dongjoon-hyun]

GitHub user dongjoon-hyun opened a pull request:

https://github.com/apache/spark/pull/11584

[SPARK-13400] Stop using deprecated Octal escape literals

## What changes were proposed in this pull request?

This removes the remaining deprecated Octal escape literals. The followings 
are the warnings on those two lines.
```
LiteralExpressionSuite.scala:99: Octal escape literals are deprecated, use 
\u instead.
HiveQlSuite.scala:74: Octal escape literals are deprecated, use \u002c 
instead.
```

## How was this patch tested?

Manual. 
During building, there should be no warning on `Octal escape literals`.
```
mvn -DskipTests clean install
```


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

$ git pull https://github.com/dongjoon-hyun/spark SPARK-13400

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

https://github.com/apache/spark/pull/11584.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 #11584


commit f7fd01ab4771537946d448c9d91e83050873a115
Author: Dongjoon Hyun 
Date:   2016-03-08T19:14:08Z

[SPARK-13400] Stop using deprecated Octal escape literals




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org