[GitHub] [arrow-ballista] thinkharderdev commented on pull request #560: Cluster state refactor part 1

2023-01-20 Thread GitBox
thinkharderdev commented on PR #560: URL: https://github.com/apache/arrow-ballista/pull/560#issuecomment-1398487737 I'll plan on merging this tomorrow morning -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [arrow-ballista] thinkharderdev commented on pull request #560: Cluster state refactor part 1

2023-01-20 Thread GitBox
thinkharderdev commented on PR #560: URL: https://github.com/apache/arrow-ballista/pull/560#issuecomment-1398473523 > I'm not as familiar with the overall Ballista architecture as I'd like to be, but this PR still makes a lot of sense. > > I _love_ seeing this stuff come out of the

[GitHub] [arrow-ballista] thinkharderdev commented on pull request #560: Cluster state refactor part 1

2022-12-19 Thread GitBox
thinkharderdev commented on PR #560: URL: https://github.com/apache/arrow-ballista/pull/560#issuecomment-1358393908 @yahoNanJing @mingmwang @andygrove Can you guys help review this? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to