You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2019/10/24 14:58:39 UTC

[GitHub] [spark] juliuszsompolski commented on issue #26221: [WIP][SPARK-29108][SQL] Add new module sql/thriftserver and add v11 thrift protocol

juliuszsompolski commented on issue #26221: [WIP][SPARK-29108][SQL] Add new module sql/thriftserver and  add v11 thrift protocol
URL: https://github.com/apache/spark/pull/26221#issuecomment-545958942
 
 
   @wangyum I agree to not commit anything the overall change is ready, but to be able to review, I would like partial PRs to be stacked on top of each other - so that the tens of thousands of lines of moving code around are in separate PRs from smaller changesets the parts that actually change something.
   My browser almost crashes when trying to read the huge PR with everything together :-).

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

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