[jira] [Resolved] (THRIFT-4282) StressTestNonBlocking is disabled in Appveyor as it is unstable on Windows in general

2020-04-23 Thread Duru Can Celasun (Jira)
[ https://issues.apache.org/jira/browse/THRIFT-4282?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Duru Can Celasun resolved THRIFT-4282. -- Fix Version/s: 0.14.0 Resolution: Resolved > StressTestNonBlocking is disabled

[GitHub] [thrift] dcelasun commented on pull request #2106: Added isOpen() to TServerTransport

2020-04-23 Thread GitBox
dcelasun commented on pull request #2106: URL: https://github.com/apache/thrift/pull/2106#issuecomment-618835805 CI is all green, merged. This is an automated message from the Apache Git Service. To respond to the message, pl

[GitHub] [thrift] dcelasun commented on pull request #2113: supplementary testcase for TJSONProtocol

2020-04-23 Thread GitBox
dcelasun commented on pull request #2113: URL: https://github.com/apache/thrift/pull/2113#issuecomment-618832363 Tests are failing: ``` File "test/thrift_json.py", line 79 if __name__ == '__main__': ^ SyntaxError: invalid syntax Makefile:6

[GitHub] [thrift] dcelasun commented on pull request #2110: THRIFT-5183: Don't try to read next frame in THeaderTransport.Read

2020-04-23 Thread GitBox
dcelasun commented on pull request #2110: URL: https://github.com/apache/thrift/pull/2110#issuecomment-618831826 Thanks! CI failures are unrelated so I'm merging this. This is an automated message from the Apache Git Service.

[GitHub] [thrift] zeshuai007 opened a new pull request #2113: supplementary testcase for TJSONProtocol

2020-04-23 Thread GitBox
zeshuai007 opened a new pull request #2113: URL: https://github.com/apache/thrift/pull/2113 - [ ] Did you create an [Apache Jira](https://issues.apache.org/jira/projects/THRIFT/issues/) ticket? (not required for trivial changes) - [ ] If a ticket exists: Does your

[GitHub] [thrift] fishy commented on pull request #2110: THRIFT-5183: Don't try to read next frame in THeaderTransport.Read

2020-04-23 Thread GitBox
fishy commented on pull request #2110: URL: https://github.com/apache/thrift/pull/2110#issuecomment-618739157 @dcelasun Test added. This is an automated message from the Apache Git Service. To respond to the message, please l

[GitHub] [thrift] dcelasun commented on pull request #2107: THRIFT-4282: Disabled StressTestNonBlocking on Windows

2020-04-23 Thread GitBox
dcelasun commented on pull request #2107: URL: https://github.com/apache/thrift/pull/2107#issuecomment-618692351 Since the original issue is more than 2 years old without any solution, I'm happy to just disable this on Windows. Restarted Travis, which was broken due to a timeout on S

[GitHub] [thrift] dcelasun commented on pull request #2106: Added isOpen() to TServerTransport

2020-04-23 Thread GitBox
dcelasun commented on pull request #2106: URL: https://github.com/apache/thrift/pull/2106#issuecomment-618691219 Travis errors seem transient, restarted the build. This is an automated message from the Apache Git Service. To

[GitHub] [thrift] dcelasun commented on pull request #1966: cpp support custom header in http client transport

2020-04-23 Thread GitBox
dcelasun commented on pull request #1966: URL: https://github.com/apache/thrift/pull/1966#issuecomment-618689998 The C++ library supports THeader, which is more or less transport agnostic and should be the preferred way of setting headers. This PR on the other hand, effectively bypas

[GitHub] [thrift] dcelasun commented on pull request #2000: THRIFT-5082: Add a Class reference for PHP enum $_TSPEC

2020-04-23 Thread GitBox
dcelasun commented on pull request #2000: URL: https://github.com/apache/thrift/pull/2000#issuecomment-618688019 Thanks for the PR @spwin! Can you please add a test case so this won't regress in the future? This is an automa

[GitHub] [thrift] dcelasun commented on pull request #2032: THRIFT-5116: Update NodeJS to supported version on Ubuntu Xenial

2020-04-23 Thread GitBox
dcelasun commented on pull request #2032: URL: https://github.com/apache/thrift/pull/2032#issuecomment-618685580 Please also update [LANGUAGES.md](https://github.com/apache/thrift/blob/master/LANGUAGES.md) and [CHANGES.md](https://github.com/apache/thrift/blob/master/CHANGES.md#0140). --

[GitHub] [thrift] dcelasun commented on pull request #2084: WIP: THRIFT-5162: ThreadManager tests fail in appveyor

2020-04-23 Thread GitBox
dcelasun commented on pull request #2084: URL: https://github.com/apache/thrift/pull/2084#issuecomment-618682028 No worries, please ping me whenever you are ready and I'll take a look. This is an automated message from the Ap

[GitHub] [thrift] dcelasun commented on pull request #2068: Supplement TSimpleFileTransport module testcase

2020-04-23 Thread GitBox
dcelasun commented on pull request #2068: URL: https://github.com/apache/thrift/pull/2068#issuecomment-618679553 This broke MSVC builds on AppVeyor, see [here](https://ci.appveyor.com/project/ApacheSoftwareFoundation/thrift/builds/32087820/job/b7qtw26hq6phy6jx?fullLog=true). Please p

[GitHub] [thrift] janosvitok commented on pull request #2084: WIP: THRIFT-5162: ThreadManager tests fail in appveyor

2020-04-23 Thread GitBox
janosvitok commented on pull request #2084: URL: https://github.com/apache/thrift/pull/2084#issuecomment-618678598 No. the code in PR causes deadlocks and I didn't have time to find out the problem, and I won't soon. Sorry.

[GitHub] [thrift] dcelasun commented on pull request #2079: THRIFT-5159: Fix runtime ReferenceError in generated code

2020-04-23 Thread GitBox
dcelasun commented on pull request #2079: URL: https://github.com/apache/thrift/pull/2079#issuecomment-618677520 Thanks for the PR @penenin! Could you please rebase from master so Travis can run again? This is an automated m

Spammy gitbox

2020-04-23 Thread Christopher
Hi Thrift Devs, Recently, it seems GitBox has been updated to start spamming the Thrift dev list. This produces a lot of redundant noise if somebody is already watching the repo on GitHub. Can these be sent to a separate notifications list for Thrift instead, so followers of the Thrift mailing lis

[GitHub] [thrift] dcelasun edited a comment on pull request #2099: supplementary testcase for TJSONProtocol

2020-04-23 Thread GitBox
dcelasun edited a comment on pull request #2099: URL: https://github.com/apache/thrift/pull/2099#issuecomment-618668928 This broke [CI](https://api.travis-ci.org/v3/job/675570277/log.txt) due to style issues. Please ping me after fixing and I'll reopen.

[GitHub] [thrift] dcelasun commented on pull request #2089: Supplementary testcase for py module

2020-04-23 Thread GitBox
dcelasun commented on pull request #2089: URL: https://github.com/apache/thrift/pull/2089#issuecomment-618676340 This broke [CI](https://api.travis-ci.org/v3/job/672411817/log.txt) due to style issues. Please ping me after fixing and I'll reopen. ---

[GitHub] [thrift] dcelasun edited a comment on pull request #2099: supplementary testcase for TJSONProtocol

2020-04-23 Thread GitBox
dcelasun edited a comment on pull request #2099: URL: https://github.com/apache/thrift/pull/2099#issuecomment-618668928 This broke [Python tests](https://api.travis-ci.org/v3/job/675570277/log.txt). Please ping me after fixing and I'll reopen. --

[GitHub] [thrift] dcelasun commented on pull request #2084: WIP: THRIFT-5162: ThreadManager tests fail in appveyor

2020-04-23 Thread GitBox
dcelasun commented on pull request #2084: URL: https://github.com/apache/thrift/pull/2084#issuecomment-618674662 @janosvitok any progress? This is an automated message from the Apache Git Service. To respond to the message, p

[GitHub] [thrift] dcelasun commented on pull request #2109: TOutput: Add further parsers for errno-to-string conversion

2020-04-23 Thread GitBox
dcelasun commented on pull request #2109: URL: https://github.com/apache/thrift/pull/2109#issuecomment-618673933 Thanks for the PR @emmenlau! I'm not familiar with Windows or MSVC but this is at least harmless for other platforms so if nobody objects, I'll go ahead and merge this in

[GitHub] [thrift] dcelasun commented on pull request #2085: fix some go default value

2020-04-23 Thread GitBox
dcelasun commented on pull request #2085: URL: https://github.com/apache/thrift/pull/2085#issuecomment-618671507 Thanks for the PR! Couple of things before this can be merged: - Please open a Jira ticket and discuss why this change is needed. It's not obvious to me why we should prea

[GitHub] [thrift] dcelasun commented on pull request #2099: supplementary testcase for TJSONProtocol

2020-04-23 Thread GitBox
dcelasun commented on pull request #2099: URL: https://github.com/apache/thrift/pull/2099#issuecomment-618668928 This broke [Python tests](https://api.travis-ci.org/v3/job/675570277/log.txt). This is an automated message fro

[GitHub] [thrift] emmenlau commented on pull request #2109: TOutput: Add further parsers for errno-to-string conversion

2020-04-23 Thread GitBox
emmenlau commented on pull request #2109: URL: https://github.com/apache/thrift/pull/2109#issuecomment-618667684 The single error in Travis CI is unrelated. Please review? This is an automated message from the Apache Git Serv

[jira] [Resolved] (THRIFT-5171) Fix maven-ant-tasks to use HTTPS instead of HTTP

2020-04-23 Thread Duru Can Celasun (Jira)
[ https://issues.apache.org/jira/browse/THRIFT-5171?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Duru Can Celasun resolved THRIFT-5171. -- Fix Version/s: 0.14.0 Resolution: Fixed > Fix maven-ant-tasks to use HTTPS inst

[GitHub] [thrift] dcelasun commented on pull request #2092: THRIFT-5171: Fix maven-ant-tasks to use HTTPS instead of HTTP

2020-04-23 Thread GitBox
dcelasun commented on pull request #2092: URL: https://github.com/apache/thrift/pull/2092#issuecomment-618666383 Thanks! This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [thrift] dcelasun commented on pull request #2110: THRIFT-5183: Don't try to read next frame in THeaderTransport.Read

2020-04-23 Thread GitBox
dcelasun commented on pull request #2110: URL: https://github.com/apache/thrift/pull/2110#issuecomment-618662755 Nice catch @fishy! Could you maybe add a test so this won't regress in a potential refactor down the line? Thi

[GitHub] [thrift] penenin opened a new pull request #2112: THRIFT-5184: Fix Connection header check for Firefox when establishing a WebSocket connection

2020-04-23 Thread GitBox
penenin opened a new pull request #2112: URL: https://github.com/apache/thrift/pull/2112 When establishing a WebSocket connection, Firefox sends Connection: keep-alive, Upgrade instead of just Connection: Upgrade. Check to see if Upgrade is in the header instead of checking to see if i

[jira] [Created] (THRIFT-5184) D: WebSocket Server Transport Fix for Firefox

2020-04-23 Thread James Lacey (Jira)
James Lacey created THRIFT-5184: --- Summary: D: WebSocket Server Transport Fix for Firefox Key: THRIFT-5184 URL: https://issues.apache.org/jira/browse/THRIFT-5184 Project: Thrift Issue Type: Bug

Who maintains the Node.js package?

2020-04-23 Thread Mario Emmenlauer
Hi all, I've seen repeated requests for an update of the Node.js package, see i.e. https://issues.apache.org/jira/browse/THRIFT-5039 and https://issues.apache.org/jira/browse/THRIFT-5170. I would be happy to understand if there is significant effort involved to make a new Node.js release? The d

[jira] [Commented] (THRIFT-5170) [Javascript] Node.js not working at all for Thrift 0.13.0 as published to npm

2020-04-23 Thread Mario Emmenlauer (Jira)
[ https://issues.apache.org/jira/browse/THRIFT-5170?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17090419#comment-17090419 ] Mario Emmenlauer commented on THRIFT-5170: -- I think this is a duplicate of THRI

[GitHub] [thrift] zeshuai007 opened a new pull request #2111: THRIFT-5179: Thrift compiler will generate wrong code if IDL struct's name is 'a' or 'b'

2020-04-23 Thread GitBox
zeshuai007 opened a new pull request #2111: URL: https://github.com/apache/thrift/pull/2111 - [ ] Did you create an [Apache Jira](https://issues.apache.org/jira/projects/THRIFT/issues/) ticket? (not required for trivial changes) - [ ] If a ticket exists: Does your

[GitHub] [thrift] stale[bot] commented on issue #2016: TNonblockingSSLServerTest.cpp: add missing include signal.h on Linux

2020-04-23 Thread GitBox
stale[bot] commented on issue #2016: URL: https://github.com/apache/thrift/pull/2016#issuecomment-618228349 This issue is no longer stale. Thank you for your contributions. This is an automated message from the Apache Git

[GitHub] [thrift] emmenlau commented on issue #2016: TNonblockingSSLServerTest.cpp: add missing include signal.h on Linux

2020-04-23 Thread GitBox
emmenlau commented on issue #2016: URL: https://github.com/apache/thrift/pull/2016#issuecomment-618228324 Thanks bot! I'll get back to this issue, promise! This is an automated message from the Apache Git Service. To respond