Skip to content

Reduction of extraneous compoents #2022

Merged
JaySon-Huang merged 9 commits intopingcap:masterfrom
SchrodingerZhu:odbc-reduction
Jun 2, 2021
Merged

Reduction of extraneous compoents #2022
JaySon-Huang merged 9 commits intopingcap:masterfrom
SchrodingerZhu:odbc-reduction

Conversation

@SchrodingerZhu
Copy link
Contributor

@SchrodingerZhu SchrodingerZhu commented May 29, 2021

What problem does this PR solve?

Problem Summary:

address the problem in #2019

What is changed and how it works?

What's Changed:

  • Remove related macro flags
  • Adjust CMake Detection
  • Remove related args from CLI

Related changes

None

Check List

Tests

  • Unit test
  • Integration test

Release note

No release note

Signed-off-by: SchrodingerZhu <i@zhuyi.fan>
Signed-off-by: SchrodingerZhu <i@zhuyi.fan>
Signed-off-by: SchrodingerZhu <i@zhuyi.fan>
Signed-off-by: SchrodingerZhu <i@zhuyi.fan>
@purelind
Copy link
Contributor

/run-all-tests

Copy link
Contributor

@JaySon-Huang JaySon-Huang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ti-srebot ti-srebot added the status/LGT1 Indicates that a PR has LGTM 1. label May 31, 2021
@JaySon-Huang
Copy link
Contributor

JaySon-Huang commented May 31, 2021

I hear that @tisonkun is planning to design federal querying for TiDB, which may need similar features.
But still, I prefer to remove these codes because:

  • they are outdated
  • TiDB/TiFlash's design may not be the same as Clickhouse

@tisonkun What's your opinion of removing these codes?

@tisonkun
Copy link
Contributor

@JaySon-Huang at least it is not yet to be implemented. I have no objection on this change.

Signed-off-by: SchrodingerZhu <i@zhuyi.fan>
Signed-off-by: SchrodingerZhu <i@zhuyi.fan>
@SchrodingerZhu SchrodingerZhu changed the title Odbc reduction Reduction of extraneous compoents Jun 1, 2021
@SchrodingerZhu
Copy link
Contributor Author

/run-all-tests

2 similar comments
@SchrodingerZhu
Copy link
Contributor Author

/run-all-tests

@purelind
Copy link
Contributor

purelind commented Jun 1, 2021

/run-all-tests

Copy link
Contributor

@JaySon-Huang JaySon-Huang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

I think we can keep the release note to be "No release note" since it doesn't matter for endpoint users. We mainly write the release note for endpoint users but not developers.

@JaySon-Huang
Copy link
Contributor

/run-all-tests

1 similar comment
@SchrodingerZhu
Copy link
Contributor Author

/run-all-tests

@JaySon-Huang
Copy link
Contributor

/rebuild

@SchrodingerZhu
Copy link
Contributor Author

/run-all-tests

@JaySon-Huang JaySon-Huang merged commit 5d3d12d into pingcap:master Jun 2, 2021
@JaySon-Huang JaySon-Huang deleted the odbc-reduction branch June 2, 2021 03:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status/LGT1 Indicates that a PR has LGTM 1.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants