Fokko commented on code in PR #2588:
URL: https://github.com/apache/thrift/pull/2588#discussion_r1169796652
##########
lib/py/setup.py:
##########
@@ -80,29 +82,30 @@ def run_setup(with_binary):
if with_binary:
extensions = dict(
ext_modules=[
- Extension('thrift.protocol.fastbinary',
- extra_compile_args=['-std=c++11'],
- sources=[
- 'src/ext/module.cpp',
- 'src/ext/types.cpp',
- 'src/ext/binary.cpp',
- 'src/ext/compact.cpp',
- ],
- include_dirs=include_dirs,
- )
+ Extension(
+ "thrift.protocol.fastbinary",
+ extra_compile_args=["-std=c++11"],
+ sources=[
+ "src/ext/module.cpp",
+ "src/ext/types.cpp",
+ "src/ext/binary.cpp",
+ "src/ext/compact.cpp",
+ ],
+ include_dirs=include_dirs,
+ )
],
- cmdclass=dict(build_ext=ve_build_ext)
+ cmdclass=dict(build_ext=ve_build_ext),
)
else:
extensions = dict()
ssl_deps = []
if sys.version_info[0] == 2:
Review Comment:
This, and also the if below can be removed.
##########
lib/py/src/TMultiplexedProcessor.py:
##########
@@ -17,9 +17,9 @@
# under the License.
#
-from thrift.Thrift import TProcessor, TMessageType
-from thrift.protocol import TProtocolDecorator, TMultiplexedProtocol
-from thrift.protocol.TProtocol import TProtocolException
+from .thrift.Thrift import TProcessor, TMessageType
Review Comment:
Was this change needed? I'm not a fan of relative imports because.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
