[GitHub] incubator-apex-core pull request: APEX-181 expose getting of rm we...

2015-10-05 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/103#discussion_r41224481 --- Diff: engine/src/main/java/com/datatorrent/stram/client/StramClientUtils.java --- @@ -732,4 +734,33 @@ public static ApplicationReport ge

[GitHub] incubator-apex-core pull request: APEX-181 expose getting of rm we...

2015-10-05 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/103#discussion_r41224451 --- Diff: engine/src/main/java/com/datatorrent/stram/security/StramWSFilterInitializer.java --- @@ -100,51 +97,20 @@ public String getProxyHo

Re: Targeting first Apex release in incubator - 3.2.0

2015-10-05 Thread Thomas Weise
The release-3.2 branch is created. We will work on getting the RC ready over the next few days. The devel-3 branch is now free for 3.3.0 work. Thanks, Thomas On Sun, Sep 27, 2015 at 10:01 PM, Thomas Weise wrote: > Just a reminder, the goal is to create the release branch for 3.2 end of > next

Re: HDFS Exception Help

2015-10-05 Thread Pramod Immaneni
Chris Nauroth could you please comment on the analysis. Thanks On Mon, Oct 5, 2015 at 7:47 PM, Ashwin Chandra Putta < ashwinchand...@gmail.com> wrote: > Tim and I explored the DFSClient code and it seems like the client got an > ack for a sequence number that has already received acknowledgement

[GitHub] incubator-apex-core pull request: APEX-181 expose getting of rm we...

2015-10-05 Thread PramodSSImmaneni
Github user PramodSSImmaneni commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/103#discussion_r41222951 --- Diff: engine/src/main/java/com/datatorrent/stram/security/StramWSFilterInitializer.java --- @@ -100,51 +97,20 @@ public String getP

[GitHub] incubator-apex-core pull request: APEX-181 expose getting of rm we...

2015-10-05 Thread PramodSSImmaneni
Github user PramodSSImmaneni commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/103#discussion_r41222832 --- Diff: engine/src/main/java/com/datatorrent/stram/client/StramClientUtils.java --- @@ -732,4 +734,33 @@ public static ApplicationRepo

Re: HDFS Exception Help

2015-10-05 Thread Ashwin Chandra Putta
Tim and I explored the DFSClient code and it seems like the client got an ack for a sequence number that has already received acknowledgement. After this warning, DFSClient should continue writing to the next datanode in the pipeline without any problem. Regards, Ashwin. On Mon, Oct 5, 2015 at 12

[GitHub] incubator-apex-core pull request: APEX-132 #resolve added Apache c...

2015-10-05 Thread vrozov
Github user vrozov commented on the pull request: https://github.com/apache/incubator-apex-core/pull/104#issuecomment-145726929 It was not meant to replace license:format or license:check. The purpose of the template is to help automate adding copyright notice to new .java/.xml/.prop

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread vrozov
Github user vrozov commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145724107 It depends on other IDE capabilities. Why to go with the least common functionality? --- If your project is set up for it, you can reply to this email and

Re: [GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread Chandni Singh
Hi All, Please provide your feedback on this discussion. Thanks, Chandni On Mon, Oct 5, 2015 at 7:12 PM, chandnisingh wrote: > Github user chandnisingh commented on the pull request: > > > https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145722960 > > Moreover I find it

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread chandnisingh
Github user chandnisingh commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145722960 Moreover I find it very odd that we add .idea folders to our repository and just completely ignore other IDEs and their metadata information. --- If

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread chandnisingh
Github user chandnisingh commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145722670 Plugins can modify the current used codeStyleSettings.xml being used. Also the indentations of 2 is consistent for xml and json so that has an effe

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread vrozov
Github user vrozov commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145720327 Plugin configuration will go into a separate file, user specific code styles to user preferences. Only Apex project code style settings are saved in the cod

[GitHub] incubator-apex-core pull request: APEX-132 #resolve added Apache c...

2015-10-05 Thread tweise
Github user tweise commented on the pull request: https://github.com/apache/incubator-apex-core/pull/104#issuecomment-145719603 The source of truth is license:format, including the exclusions. And license:check will be part of the CI. Are you sure you can replicate all of that for In

[GitHub] incubator-apex-core pull request: APEX-132 #resolve added Apache c...

2015-10-05 Thread vrozov
Github user vrozov commented on the pull request: https://github.com/apache/incubator-apex-core/pull/104#issuecomment-145717871 The template is not .java specific. It defines template and IntelliJ knows how to apply the template to different file types (properties, xml and etc).

[GitHub] incubator-apex-core pull request: APEX-132 #resolve added Apache c...

2015-10-05 Thread tweise
Github user tweise commented on the pull request: https://github.com/apache/incubator-apex-core/pull/104#issuecomment-145716392 The issue of tracking files under the .idea folder is discussed in #102 Why to solve this on the IDE level when license:format will take care of it,

[GitHub] incubator-apex-core pull request: APEX-132 #resolve added Apache c...

2015-10-05 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/incubator-apex-core/pull/104 APEX-132 #resolve added Apache copyright template for IntelliJ @tweise, @PramodSSImmaneni please review You can merge this pull request into a Git repository by running: $ git pull

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread chandnisingh
Github user chandnisingh commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145713286 There are plugins that may add to codeStyleSettings in the users .idea folder. This will result in conflicts every time the user pulls. --- If your

[GitHub] incubator-apex-core pull request: APEX-181 expose getting of rm we...

2015-10-05 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/incubator-apex-core/pull/103 APEX-181 expose getting of rm web app address @PramodSSImmaneni I need to expose the function of getting all RM webapp addresses when HA is enabled so I'm moving some of the code to

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread tweise
Github user tweise commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145694805 @siyuanh This does not apply to a Maven project. In any case, we don't want to check in files that can be generated. --- If your project is set up for it,

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread vrozov
Github user vrozov commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145693952 Is there a plugin for IntelliJ to generate codeStyleSettings.xml? --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread siyuanh
Github user siyuanh commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145693264 I don't recommend checkin any IDE specific files either. We can use maven plugins to generate IDE setting files if user wants. Ex. mvn eclipse:eclipse ---

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread vrozov
Github user vrozov commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145693123 This should not be a problem. Workspace and other files don't need to be added to the source control. Having .idea under source control will not delete othe

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread chandnisingh
Github user chandnisingh commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145692498 The .idea folder under user's project workspace has many other user based settings. These settings varies with different plugins installed by users, p

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread vrozov
Github user vrozov commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145691533 Please see https://intellij-support.jetbrains.com/hc/en-us/articles/206827587-How-to-manage-projects-under-Version-Control-Systems. IMHO, the less manual st

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread chandnisingh
Github user chandnisingh commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145689519 I am following the IntelliJ recommended way of sharing the code style which is why added codeStyleSettings.xml as per your recommendation. The users ne

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread siyuanh
Github user siyuanh commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145689141 We need gradle :) --- 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

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread vrozov
Github user vrozov commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145688326 I can't say for other IDEs, for IntelliJ this is IDE recommended way to share code style (and few other artifacts like copyright) for a project using source

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread chandnisingh
Github user chandnisingh commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145686830 Only the code style not all the other settings so if we create .idea folder inside APEX then that will create problems with users workspace. We don't

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread vrozov
Github user vrozov commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145686181 It is intended to overwrite user code style settings for the apex project. --- If your project is set up for it, you can reply to this email and have your r

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread chandnisingh
Github user chandnisingh commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145685909 Checking in .idea folder will imply over-writing users project settings which is not intended. We require users to only use apex style not modify all

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread vrozov
Github user vrozov commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145685496 What is the reason? --- 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

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread chandnisingh
Github user chandnisingh commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145685012 we can't check-in a .idea folder --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If yo

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread vrozov
Github user vrozov commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145684744 Please move codeStyleSettings.xml from misc/ide-templates/intellij/ to .idea. --- If your project is set up for it, you can reply to this email and have y

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread chandnisingh
Github user chandnisingh commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145678156 Made the changes suggested by @vrozov and updated README. please review and merge --- If your project is set up for it, you can reply to this email a

Re: [GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread Vlad Rozov
Resending to dev@apex.incubator.apache.org as github does not support .xml for attachments. Please try with the attached file. Copy it to the .idea directory. Thank you, Vlad On 10/5/15 12:54, chandnisingh wrote: Github user chandnisingh commented on the pull request: https://github.c

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread vrozov
Github user vrozov commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145655907 Please try attached file. Copy it to .idea and select project code style. On 10/5/15 12:54, Chandni Singh wrote: > > @vrozov

HDFS Exception Help

2015-10-05 Thread Timothy Farkas
Hello All, I am seeing this exception in one of my applications, and I am wondering if there are any explanations for why this is happening or if there are any solutions? Any help would be appreciated. 2015-10-02 11:43:23,951 WARN org.apache.hadoop.hdfs.DFSClient: DFSOutputStream ResponseProcesso

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread chandnisingh
Github user chandnisingh commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145649360 @vrozov tried it and it doesn't work for code styles. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread vrozov
Github user vrozov commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145647791 For IntelliJ, codeStyleSettings.xml can be added to .idea subdirectory. We may also add Apache copyright as well. Please see https://intellij-support.jetbra

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread chandnisingh
Github user chandnisingh commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145620656 Added links to instructions for individual IDEs from main project README. Please review and merge. Since the check style changes are going in 3.2.0, it

Re: October Apex Incubator Report

2015-10-05 Thread Alan Gates
What does "Grow the community beyond Jira and ML participation." mean? Do you mean you want people who are currently just doing mailing lists and JIRA to contribute patches too? There will always be many more mailing list and JIRA participants than coding ones, and that's not a bad thing. Ma

[GitHub] incubator-apex-core pull request: APEX-157 #comment #resolve Added...

2015-10-05 Thread ishark
Github user ishark commented on the pull request: https://github.com/apache/incubator-apex-core/pull/81#issuecomment-145607263 Code changes have been merged with devel-3. Closing the pull request. --- If your project is set up for it, you can reply to this email and have your reply ap

[GitHub] incubator-apex-core pull request: APEX-157 #comment #resolve Added...

2015-10-05 Thread ishark
Github user ishark closed the pull request at: https://github.com/apache/incubator-apex-core/pull/81 --- 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 fea

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread chandnisingh
Github user chandnisingh commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145604626 It will only modify Java code style settings not the entire IDE settings. IntelliJ jar contains only a code style xml and NetBeans contains only th

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread PramodSSImmaneni
Github user PramodSSImmaneni commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145604146 Chandni if somebody imports these settings will it just modify the java settings or will it replace the entire IDE settings for the user. --- If

[GitHub] incubator-apex-malhar pull request: Cleanup README.md

2015-10-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-apex-malhar/pull/53 --- 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 f

[GitHub] incubator-apex-malhar pull request: Cleanup README.md

2015-10-05 Thread tweise
Github user tweise commented on a diff in the pull request: https://github.com/apache/incubator-apex-malhar/pull/53#discussion_r41169709 --- Diff: README.md --- @@ -1,7 +1,7 @@ -Malhar -== -Malhar repository contains open source operator and codec library that can b

[GitHub] incubator-apex-core pull request: APEX-178 #resolve #comment added...

2015-10-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-apex-core/pull/101 --- 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 fe

[GitHub] incubator-apex-malhar pull request: Cleanup README.md

2015-10-05 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-malhar/pull/53#discussion_r41165362 --- Diff: README.md --- @@ -1,7 +1,7 @@ -Malhar -== -Malhar repository contains open source operator and codec library that ca

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread chandnisingh
Github user chandnisingh commented on the pull request: https://github.com/apache/incubator-apex-core/pull/102#issuecomment-145562630 NetBeans settings are bundled as zip and Intellij as jar so I am not sure if these files can be checked in this repository. Please let me know where to

[GitHub] incubator-apex-core pull request: APEX-151 #resolve #comment added...

2015-10-05 Thread chandnisingh
GitHub user chandnisingh opened a pull request: https://github.com/apache/incubator-apex-core/pull/102 APEX-151 #resolve #comment added templates for eclipse, intellij and … …netbeans You can merge this pull request into a Git repository by running: $ git pull https://githu

Incubator PMC/Board report for Oct 2015 ([ppmc])

2015-10-05 Thread Marvin
Dear podling, This email was sent by an automated system on behalf of the Apache Incubator PMC. It is an initial reminder to give you plenty of time to prepare your quarterly board report. The board meeting is scheduled for Wed, 21 October 2015, 10:30 am PST. The report for your podling will

Obsolete Coding Conventions and Style document in Malhar

2015-10-05 Thread Chandni Singh
Hi, There is an obsolete Coding Convention and Style document in Malhar under docs folder. I think this should be removed from Malhar. I have created APEX-180 to create a style guide in APEX core. Chandni